[clang] [llvm] [clang][Inliner] Add opt-in inliner hints and implementation-file functions (PR #228074)
Nikita Popov via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 12 03:49:27 PDT 2026
https://github.com/nikic commented:
LLVM side of this looks good to me. Clang side should be reviewed by someone else -- though I'll already note that Clang is not supposed to use `cl::opt` options...
>From the LLVM side, I think the only questions I have would be:
* Should we verify that the attribute is only used with linkonce_odr linkage? Given that the attribute is non-semantic, I think not verifying is fine, and maybe preferable.
* Should this be an enum attribute instead of a string attribute? Given that it's only check in this one place I'm fine with the string attribute, but making it an enum attribute is probably more principled.
https://github.com/llvm/llvm-project/pull/228074
More information about the cfe-commits
mailing list