[PATCH] D18775: NFC: make AtomicOrdering an enum class

JF Bastien via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 4 16:24:15 PDT 2016


jfb added a comment.

Secret note: I'm also hoping that this is a small step towards making `consume` easier to experiment with. Shh don't tell anyone.


================
Comment at: include/llvm/IR/Instructions.h:64
@@ +63,3 @@
+  switch (ao) {
+  case AtomicOrdering::NotAtomic: return "not_atomic";
+  case AtomicOrdering::Unordered: return "unordered";
----------------
This never happens. I can make it unreachable if that seems better.


http://reviews.llvm.org/D18775





More information about the llvm-commits mailing list