[PATCH] D107443: AArch64: support @llvm.va_copy in GISel

[Tim Northover via Phabricator via llvm-commits]

t.p.northover closed this revision.
t.p.northover added a comment.

Thanks, committed as 5ad086089962 <https://reviews.llvm.org/rG5ad086089962c8c6374c138e297b821ed042197e>.



================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp:975
+                                           MachineMemOperand::MOLoad,
+                                           VaListSize, Align(8)));
+    MIB.buildStore(Val, MI.getOperand(1),
----------------
paquette wrote:
> SDAG sets `Align` to be the pointer size here. Should we do that too?
Oh yes, well spotted!


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

https://reviews.llvm.org/D107443