[PATCH] D157029: [llvm] Construct option's prefixed name at compile-time

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 4 12:58:24 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 cfe-commits mailing list