[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