[clang] [clang] -fstrict-enums optimization for by-value parameters (PR #154807)

via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 5 13:12:59 PDT 2025


keinflue wrote:

> Thanks, I think this looks pretty good.

Thank you.

Do you have any insights on my two comments?

> As a side effect this also applies these range attributes to types with boolean representation which are not represented as i1 in parameters (if these exist).

> AssumeBooleanRepresentation is used here because of || NSAPI(CGM.getContext()).isObjCBOOLType(Ty) in IsBool above. This condition was not present in the use of getRangeForType for load optimization purposes and I do not know why it is special cased like this for the sanitizer.

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


More information about the cfe-commits mailing list