[PATCH] D43618: [X86][MMX] Add support for MMX build vectors (PR29222)

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:30863
+    if (V.getValueType().isFloatingPoint()) {
+      if (Subtarget.hasSSE1() && Subtarget.is64Bit() &&
+          !isa<ConstantFPSDNode>(V)) {
----------------
Why does this require 64-bit?


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:30865
+          !isa<ConstantFPSDNode>(V)) {
+        V = DAG.getNode(ISD::INSERT_VECTOR_ELT, DL, MVT::v4f32,
+                        DAG.getUNDEF(MVT::v4f32), V,
----------------
Should this be SCALAR_TO_VECTOR?


Repository:
  rL LLVM

https://reviews.llvm.org/D43618