[PATCH] D15132: [X86][SSE] Update the cost table for integer-integer conversions on SSE2.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 07:04:51 PST 2015


RKSimon added inline comments.

================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:703
@@ +702,3 @@
+    { ISD::ZERO_EXTEND, MVT::v4i16,  MVT::v4i8,   1 },
+    { ISD::SIGN_EXTEND, MVT::v4i16,  MVT::v4i8,   6 },
+
----------------
These values don't appear to be correct for SSE41 which has PMOVSX/PMOVZX ops - maybe split off the 128-bit extension ops from AVXConversionTbl into SSE41ConversionTbl ?


http://reviews.llvm.org/D15132





More information about the llvm-commits mailing list