[clang] [llvm] [SpecialCaseList] Use glob by default (PR #74809)
Fangrui Song via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 15 15:48:38 PDT 2024
MaskRay wrote:
> CC @llvm/clang-vendors
>
> > I [announced this change on discourse](https://discourse.llvm.org/t/use-glob-instead-of-regex-for-specialcaselists/71666?u=ellishg) last year.
>
> "Use glob instead of regex for SpecialCaseLists" doesn't mean anything for anyone not actively working on the relevant code. An announcement would be titled something like "Syntax change for -fsanitize-ignorelist"... and it would be posted when the change actually happened.
Apologies. This could have been better handled. I've also seen a report of `[cfi-vcall|cfi-nvcall|cfi-icall]` from Chromium. Now I re-checked, https://reviews.llvm.org/D154014 did mention `[cfi-vcall|cfi-icall]`. Grepping `\|` (and `^\[.*\|` for section names) in `*san_ignorelist.txt` files is probably good enough to know whether you affected.
https://github.com/llvm/llvm-project/pull/74809
More information about the cfe-commits
mailing list