[PATCH] D11413: X86: Fixed assertion failure in 32-bit mode

Michael Kuperstein michael.m.kuperstein at intel.com
Wed Jul 22 07:09:27 PDT 2015

mkuper added inline comments.

Comment at: lib/Target/X86/X86ISelLowering.cpp:7385
@@ -7385,1 +7384,3 @@
+                                               DAG);
+  if (V2S && DAG.getTargetLoweringInfo().isTypeLegal(V2S.getValueType())) {
     // We need to zext the scalar if it is smaller than an i32.
This seems a bit odd - if getScalarValueForVectorElement() returned a non-null value, there was already a SCALAR_TO_VECTOR or a BUILD_VECTOR there. So why would forming a SCALAR_TO_VECTOR be wrong?

(I'm not saying it's not, I just don't understand why.)



More information about the llvm-commits mailing list