[PATCH] prevent folding a scalar FP load into a packed logical FP instruction (PR22371)

[Quentin Colombet]

Hi Sanjay,


================
Comment at: lib/Target/X86/X86InstrFragmentsSIMD.td:374
@@ +373,3 @@
+def loadf32_128 : PatFrag<(ops node:$ptr),
+  (bitconvert (v4f32 (scalar_to_vector (loadf32 node:$ptr))))>;
+def loadf64_128 : PatFrag<(ops node:$ptr),
----------------
This is not valid, is it?

When this matches we will read 128-bit from the memory, i.e., pass what we do for the load32. Aren’t we?

Something correct would be load128 -> extract element.
Though I do not think that happens a lot…

http://reviews.llvm.org/D7474

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/