[PATCH] D31521: DAG: Fix missing legalization for any_extend_vector_inreg operands

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 3 06:03:22 PDT 2017


RKSimon added a subscriber: llvm-commits.
RKSimon added a comment.

Please don't forgot to subscribe cfe/llvm-commits on your tickets.



================
Comment at: lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:1681
+  SDValue Lo, Hi;
+  SplitVecRes_ExtVecInRegOp(N, Lo, Hi);
+
----------------
We don't normally do this so I think you need to comment this properly - explaining that *_EXTEND_VECTOR_INREG only reference the lower half of the input and that splitting the result has the same effect as splitting the input operand.


https://reviews.llvm.org/D31521





More information about the llvm-commits mailing list