[PATCH] D57139: GlobalISel: Fix creating MMOs with align 0
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 25 10:49:58 PST 2019
arsenm marked an inline comment as done.
arsenm added inline comments.
================
Comment at: lib/CodeGen/GlobalISel/IRTranslator.cpp:854
.addMemOperand(MF->getMachineMemOperand(
- MachinePointerInfo(Ptr), MachineMemOperand::MOStore, ListSize, 0));
+ MachinePointerInfo(Ptr), MachineMemOperand::MOStore, ListSize, 1));
return true;
----------------
aemerson wrote:
> Can we use DL.getABITypeAlignment() here?
Probably? I don't know anything about vastart. I'm not sure why this has an MMO in the first place
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57139/new/
https://reviews.llvm.org/D57139
More information about the llvm-commits
mailing list