[Mlir-commits] [mlir] [mlir][Tosa] Add unreachable case for bad Extension type in TosaProfileCompliance (PR #128889)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Wed Feb 26 08:10:28 PST 2025


================
@@ -148,6 +148,7 @@ class TosaProfileCompliance {
     case Extension::none:
----------------
enkerewpo wrote:

Yeah, default: llvm_unreachable(...) and placing llvm_unreachable at the end of the function are semantically identical and both are commonly used in the LLVM codebase. Since MLIR code was using the `default` style, I’ll go with that approach.

https://github.com/llvm/llvm-project/pull/128889


More information about the Mlir-commits mailing list