[PATCH] D60762: [SelectionDAG] Legalize vaargs that require vector splitting

[ChenZheng via Phabricator via llvm-commits]

shchenz added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:1858
+  Lo = DAG.getVAArg(NVT, dl, Chain, Ptr, SV, Align);
+  Hi = DAG.getVAArg(NVT, dl, Lo.getValue(1), Ptr, SV, 0);
+  Chain = Hi.getValue(1);
----------------
Why `vaarg` has alignment `0`? I think it should be the result of `getABITypeAlignment()` for type of `Hi`? and also the alignment for `Lo` should not be right.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60762/new/

https://reviews.llvm.org/D60762