[PATCH] D138458: [Clang][Driver] Add KCFI to SupportsCoverage
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 21 18:23:58 PST 2022
MaskRay added inline comments.
================
Comment at: clang/test/CodeGen/sanitize-coverage.c:94
+ // CHECK-NOT: call void @__sanitizer_cov_trace_const_cmp
+ // KCFI-NOT: call void %{{.}}() [ "kcfi"(i32 {{.*}}) ]
+ f();
----------------
samitolvanen wrote:
> MaskRay wrote:
> > `%c()`
> >
> > `{{.}}` matches one byte. Just write the byte.
> > `%c()`
> >
> > `{{.}}` matches one byte. Just write the byte.
>
> I do want to match just one byte, but the actual byte changes depending on how many statements there are in the function, so matching any byte seems less fragile. `%c` would match literally just `%c`. Am I missing some neat FileCheck trick here that would save me a ton of typing in future...?
Ah, ok, it's an unnamed value. `[[#]]` can match `%1` `%2` ...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138458/new/
https://reviews.llvm.org/D138458
More information about the cfe-commits
mailing list