[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