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

[Konstantin Zhuravlyov via llvm-commits]

kzhuravl added inline comments.

================
Comment at: include/llvm/CodeGen/MachineMemOperand.h:123
@@ +122,3 @@
+    /// Synchronization scope for this memory operation.
+    unsigned SynchScope : 1;      // enum SynchronizationScope
+    /// Atomic ordering requirements for this memory operation. For cmpxchg
----------------
If this patch (or some variation of it) is acceptable, the size of SynchScope will increase to 8 bits. Furthermore, we can generalize `MachineAtomicInfo` to `AdditionalData` with extra 16 bits available.


https://reviews.llvm.org/D24577