thurstond wrote:

> Yeah, agreed, alloca+store+load per candidate isn't "small runtime cost, no 
> ABI impact." That's just not the UBSan contract. Easiest fix: pull 
> `uninitialized-read` out of the default `Undefined` group, make it opt-in 
> like `unsigned-integer-overflow` already is. Keeps the check, drops the 
> "changes what `-fsanitize=undefined` costs you by default" problem.

That doesn't fix it: the quote about "small runtime cost and no impact on 
address space layout or ABI" applies to all the UBSan checks, even the opt-in 
ones.

https://github.com/llvm/llvm-project/pull/207529
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to