Optimize x86 vector shift by constant.

BUG=swiftshader:15 Change-Id: I4b7b97f3de18c201a502d0bc38a2c845a1caf278 Reviewed-on: https://chromium-review.googlesource.com/392627Tested-by: Nicolas Capens <nicolascapens@google.com> Reviewed-by: Jim Stichnoth <stichnot@chromium.org>

Optimize x86 vector shift by constant.
b0935390 · Nicolas Capens · Nicolas Capens · 71c69373 · b0935390
Commit b0935390 authored Oct 04, 2016 by Nicolas Capens Committed by Nicolas Capens Oct 07, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 24 additions and 3 deletions

IceTargetLoweringX86BaseImpl.h src/IceTargetLoweringX86BaseImpl.h +24 -3

No files found.
--- a/src/IceTargetLoweringX86BaseImpl.h
+++ b/src/IceTargetLoweringX86BaseImpl.h
@@ -2142,9 +2142,27 @@ void TargetX86Base<TraitsType>::lowerArithmetic(const InstArithmetic *Instr) {
        llvm::report_fatal_error("Invalid vector multiply type");
      }
    } break;
-    case InstArithmetic::Shl:
+    case InstArithmetic::Shl: {
-    case InstArithmetic::Lshr:
+      assert(llvm::isa<Constant>(Src1) && "Non-constant shift not scalarized");
-    case InstArithmetic::Ashr:
+      Variable *T = makeReg(Ty);
+      _movp(T, Src0);
+      _psll(T, Src1);
+      _movp(Dest, T);
+    } break;
+    case InstArithmetic::Lshr: {
+      assert(llvm::isa<Constant>(Src1) && "Non-constant shift not scalarized");
+      Variable *T = makeReg(Ty);
+      _movp(T, Src0);
+      _psrl(T, Src1);
+      _movp(Dest, T);
+    } break;
+    case InstArithmetic::Ashr: {
+      assert(llvm::isa<Constant>(Src1) && "Non-constant shift not scalarized");
+      Variable *T = makeReg(Ty);
+      _movp(T, Src0);
+      _psra(T, Src1);
+      _movp(Dest, T);
+    } break;
    case InstArithmetic::Udiv:
    case InstArithmetic::Urem:
    case InstArithmetic::Sdiv:
@@ -7009,6 +7027,9 @@ void TargetX86Base<TraitsType>::genTargetHelperCallFor(Inst *Instr) {
      case InstArithmetic::Shl:
      case InstArithmetic::Lshr:
      case InstArithmetic::Ashr:
+        if (llvm::isa<Constant>(Src1)) {
+          return;
+        }
      case InstArithmetic::Udiv:
      case InstArithmetic::Urem:
      case InstArithmetic::Sdiv: