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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to