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

[Justin Lebar via llvm-commits]

jlebar added inline comments.

================
Comment at: include/llvm/CodeGen/MachineMemOperand.h:128
@@ -124,1 +127,3 @@
+  AtomicOrdering Ordering;
+  AtomicOrdering FailureOrdering;
 
----------------
I am concerned that this will *substantially* increase the size of MMO.

Do you actually need all this space?


https://reviews.llvm.org/D24577