[PATCH] [x86] Add a ZERO_EXTEND_VECTOR_INREG DAG node and use it when widening vector types to be legal and a ZERO_EXTEND node is encountered.

Filipe Cabecinhas filcab+llvm.phabricator at gmail.com
Mon Jul 7 16:18:17 PDT 2014


LGTM after the legality check gets taken care of (either because we already check the resulting node or by recursing manually).

================
Comment at: lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:2420
@@ +2419,3 @@
+  // If some legalization strategy other than widening is used on the operand,
+  // we can't safely assume that just zero-extending the low lanes in the
+  // correct transformation.
----------------
s/in the/is the/?

http://reviews.llvm.org/D4405






More information about the llvm-commits mailing list