[PATCH] D157029: [llvm] Construct option's prefixed name at compile-time
Jan Svoboda via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 4 12:58:23 PDT 2023
jansvoboda11 added a comment.
In D157029#4561519 <https://reviews.llvm.org/D157029#4561519>, @jansvoboda11 wrote:
> In D157029#4561490 <https://reviews.llvm.org/D157029#4561490>, @MaskRay wrote:
>
>> This increases the size of `Info` (static data size and static relocations). In return, some dynamic relocations are saved. Is this a net win?
>
> If that's a concern, I can remove `Info::Name` and replace its usages with a function call that drops the prefix from prefixed name.
Done.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157029/new/
https://reviews.llvm.org/D157029
More information about the llvm-commits
mailing list