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

James Y Knight via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 6 11:59:29 PDT 2016


jyknight added a comment.

If we're having this churn, I'd at least like to get some safety out of it. :) Looks like = delete would work:

  bool operator<(AtomicOrdering, AtomicOrdering) = delete;
  bool operator>(AtomicOrdering, AtomicOrdering) = delete;
  bool operator<=(AtomicOrdering, AtomicOrdering) = delete;
  bool operator>=(AtomicOrdering, AtomicOrdering) = delete;


http://reviews.llvm.org/D18775





More information about the llvm-commits mailing list