[clang] Add support for builtin_verbose_trap (PR #79230)
Akira Hatanaka via cfe-commits
cfe-commits at lists.llvm.org
Fri May 10 11:02:00 PDT 2024
================
@@ -172,6 +172,27 @@ static bool checkArgCount(Sema &S, CallExpr *Call, unsigned DesiredArgCount) {
<< /*is non object*/ 0 << Call->getArg(1)->getSourceRange();
}
+static bool checkBuiltinVerboseTrap(CallExpr *Call, Sema &S) {
----------------
ahatanak wrote:
clang already checks that the number of arguments is exactly 2 (see `test/SemaCXX/verbose-trap.cpp`).
I agree that we should check that `$` isn't in the strings.
https://github.com/llvm/llvm-project/pull/79230
More information about the cfe-commits
mailing list