[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.
Matt Arsenault
Matthew.Arsenault at amd.com
Mon Jul 7 11:27:25 PDT 2014
================
Comment at: lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp:750
@@ +749,3 @@
+ return DAG.getNode(ISD::BITCAST, DL, VT,
+ DAG.getVectorShuffle(SrcVT, DL, Zero, Src, ShuffleMask));
+}
----------------
Benjamin Kramer wrote:
> Can we get away with creating illegal shuffles here or should we ask the backend if the mask is legal first?
I'm pretty sure this needs to be checked. I fixed a similar issue in http://reviews.llvm.org/D4320
http://reviews.llvm.org/D4405
More information about the llvm-commits
mailing list