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

Brandon Wu via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 22 10:39:09 PDT 2024


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

I guess it won't get into `Type::Record` if it fall through from `Type::Builtin`, however I agree with you the intention of the code snippet.
Let me add a default case which is unreachable.

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


More information about the cfe-commits mailing list