[PATCH] D60762: [SelectionDAG] Legalize vaargs that require vector splitting
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 21 12:48:40 PDT 2019
efriedma added inline comments.
================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:1860
+ if (DAG.getDataLayout().isBigEndian())
+ std::swap(Lo, Hi);
+
----------------
This probably doesn't match how the vector will actually be passed on a big-endian target. Endianness doesn't affect the ordering of the elements of a vector in LLVM. (See also http://llvm.org/docs/BigEndianNEON.html .)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60762/new/
https://reviews.llvm.org/D60762
More information about the llvm-commits
mailing list