[clang] [Clang] Overflow Idiom Exclusions (PR #100272)

Justin Stitt via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 23 16:46:19 PDT 2024


JustinStitt wrote:

> > Right, unsigned overflow is well-defined. This isn't about the semantics of "undefined-ness". We want the unsigned sanitizer ON but some of the things it reports are noisy -- that's what this PR is about
> 
> "Idiom" seems to vague to me. kernel has own opinion on what is ideom, other users may have different ideoms.

What's a better word? Pattern?

> 
> Also so far we were able to avoid behavior tweaking flags in UBSAN. like `-fno-sanitize-overflow-idioms` (e.g. we have them in Asan). In UBSAN we were able to express as sets of checks. It would be nice to continue these way.
> 
> WDYT on splitting existing checks into smaller checks with verbose naming?

Yes, that's a good idea. Doing it that way, we can add more "idioms" in the future and users can opt into those ones.

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


More information about the cfe-commits mailing list