[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