[clang] [ASTContext] Add a break to nested switch in `encodeTypeForFunctionPointerAuth` (PR #99763)

Paul Kirth via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 22 10:42:01 PDT 2024


================
@@ -3363,6 +3363,7 @@ static void encodeTypeForFunctionPointerAuth(const ASTContext &Ctx,
 #include "clang/Basic/RISCVVTypes.def"
       llvm_unreachable("not yet implemented");
     }
+    break;
----------------
ilovepi wrote:

won't you get an error for having a `default` case on a fully covered switch? Plus that will remain working if someone adds a new enum value, but doesn't update this code, right?

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


More information about the cfe-commits mailing list