[PATCH] D11886: [InstCombine] Move SSE2/AVX2 arithmetic vector shift folding to instcombiner

[Michael Kuperstein via llvm-commits]

mkuper added a comment.

Thanks Simon,

I have a few minor questions about testing, other than that, looks good.


================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:261
@@ -253,1 +260,3 @@
+
+  return Builder.CreateAShr(Vec, ShiftVec);
 }
----------------
Do we have an ISel test that these AShrs get lowered correctly? If we don't, should we?

================
Comment at: test/CodeGen/X86/combine-avx2-intrinsics.ll:6
@@ -5,3 +5,1 @@
 
-define <8 x i32> @test_psra_1(<8 x i32> %A) {
-  %1 = tail call <8 x i32> @llvm.x86.avx2.psrai.d(<8 x i32> %A, i32 3)
----------------
We still want to test these combines, right? (Only as part of InstCombine, not ISel)


Repository:
  rL LLVM

http://reviews.llvm.org/D11886