[PATCH] D141317: [SDAG] Introduce a common MEMBARRIER node [nfc]

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Target/X86/X86InstrCompiler.td:687
                      "#MEMBARRIER",
-                     [(X86MemBarrier)]>, Sched<[WriteLoad]>;
+                     [(membarrier)]>, Sched<[WriteLoad]>;
 
----------------
Why not go one step further and have a generic barrier machine instr?


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

https://reviews.llvm.org/D141317