[PATCH] D118681: [AArch64] Do not use ABI alignment for mops.memset.tag

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 1 03:42:51 PST 2022


dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

Hmm. I was expecting to be able to test this by printing the MIR and inspecting the operand, but it appears that the alignment gets lost as we convert to a machine node. I'm not sure why.

Like you said the alignment won't be used for much though, so I think this LGTM as-is. So long as we have a test that the align operand isn't doing anything funny, that Ok on its own. Thanks.


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

https://reviews.llvm.org/D118681



More information about the llvm-commits mailing list