[PATCH] D14901: [X86][SSE] Improve i16 splatting shuffles

[Quentin Colombet via llvm-commits]

qcolombet added inline comments.

================
Comment at: test/CodeGen/X86/avx-splat.ll:18
@@ -18,1 +17,3 @@
+; CHECK-NEXT:    vpshufhw {{.*#+}} xmm0 = xmm0[0,1,2,3,5,5,5,5]
+; CHECK-NEXT:    vpshufd {{.*#+}} xmm0 = xmm0[2,2,3,3]
 ; CHECK-NEXT:    vinsertf128 $1, %xmm0, %ymm0, %ymm0
----------------
I am missing something.

You said:
> Note this has the side effect of a lot of SSSE3 test cases no longer needing to use PSHUFB, as it falls below the 3 op combine threshold for when PSHUFB is typically worth it.

But isn’t pshufb always better than two other pshufX instructions?


Repository:
  rL LLVM

http://reviews.llvm.org/D14901