================
@@ -290,6 +290,7 @@ static llvm::AtomicOrdering mapCABIAtomicOrdering(unsigned 
AO) {
   case llvm::AtomicOrderingCABI::relaxed:
     return llvm::AtomicOrdering::Monotonic;
   }
+  llvm_unreachable("Unknown AtomicOrderingCABI enum");
----------------
Thibault-Monnier wrote:

It seems to me there is only a warning when enabling `-Wswitch-enum`, but then, 
that's basically the point of the flag.

What gcc _does_ do is that it never considers a `switch case` without a 
`default` as handling all control paths, see https://godbolt.org/z/E8PcGoPv6. 
However, that's not applicable in this case. 

https://github.com/llvm/llvm-project/pull/180500
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to