[clang] [compiler-rt] [Sanitizer] add signed-integer-wrap sanitizer (PR #80089)

Nick Desaulniers via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 13 10:41:24 PST 2024


https://github.com/nickdesaulniers commented:

PATCH mostly LGTM

UBSan is documented clang/docs/UndefinedBehaviorSanitizer.rst https://clang.llvm.org/docs/UndefinedBehaviorSanitizer.html

Please make sure to update the list [of available checks](https://clang.llvm.org/docs/UndefinedBehaviorSanitizer.html#available-checks).

Please also update the release notes to mention this feature.
clang/docs/ReleaseNotes.rst

> The kernel enables -fno-strict-overflow which implies -fwrapv.

While Linux is the only kernel I've contributed to, it's important to be explicit as to which kernel you're referring to. 😛 

> and likely in production kernels. :)

I doubt you meant running in production. 😛 

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


More information about the cfe-commits mailing list