[PATCH] D24577: [RFC] Move synchronization scope and atomic orderings from SDNode to MachineMemOperand

[Tom Stellard via llvm-commits]

tstellarAMD added a comment.

In https://reviews.llvm.org/D24577#557734, @jlebar wrote:

> In https://reviews.llvm.org/D24577#548365, @kzhuravl wrote:
>
> > Currently machine memory operand's member layout has a 4 byte hole between `BaseAlignLog2` and `AAInfo`.
>
>
> Only on 64-bit platforms, right?  Are we willing to bite a 4-byte increase to MMO size on 32-bit?


This would be 4-bytes per MachineInstr with an MMO, that doesn't see too bad to me.


https://reviews.llvm.org/D24577