thurstond wrote:

> FWIW, I think the `cfi_` prefix is no longer a great fit, since you want it 
> to apply to multiple things. Perhaps just `unchecked_callee`? I don't have a 
> strong opinion here, but I find it a bit odd to have to add 
> `cfi_unchecked_callee` if I want to opt out of checking for UBSan. That's 
> just my 2 cents though.

Control-flow integrity violations are undefined behavior, hence CFI is somewhat 
of an honorary member of UBSan.

For `-fsanitize-annotate-debug-info=cfi`, we have the opposite naming 
convention: it will emit e.g., `__ubsan_check_cfi_icall`.

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

Reply via email to