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

Do you mean we should allow it to work if someone adds a new enum but doesn't 
update the code?
That's why I added a break rather than unreachable haha~
btw, why we get an error if having `default` case on a fully covered switch?


https://github.com/llvm/llvm-project/pull/99763
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to