[PATCH] [CodeGen] Add hooks/combine to form vector extloads, and enable it on X86.

hfinkel at anl.gov hfinkel at anl.gov
Tue Jan 27 18:37:30 PST 2015


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:5227
@@ +5226,3 @@
+
+  // fold (sext (load x)) -> (sext (truncate (sextload x)))
+  // Only on illegal but splittable vectors.
----------------
Please make this comment more verbose. We might generate multiple loads and then combine them together, and it is not clear here whether this combining happens to the results of the sext or the sext's operand.

================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:5274
@@ +5273,3 @@
+      LN0->isVolatile(), LN0->isNonTemporal(), LN0->isInvariant(),
+      LN0->getAlignment(), LN0->getAAInfo());
+
----------------
Is this alignment correct? Aren't these loads smaller than the original one?

http://reviews.llvm.org/D6904

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






More information about the llvm-commits mailing list