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

[JF Bastien via llvm-commits]

jfb added a comment.

In http://reviews.llvm.org/D18775#395975, @dberlin wrote:

> BTW, this is what i came up with, not sure if anyone has a better way
>
> inline hash_code hash_value(const AtomicOrdering &AO) {
>
>   return static_cast<typename
>
> std::underlying_type<AtomicOrdering>::type>(AO);
>  }


This should be fixed by http://reviews.llvm.org/D18938. It turns out hashing all enum classes was already *intended* to work.


Repository:
  rL LLVM

http://reviews.llvm.org/D18775