[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:10:30 PDT 2023


jansvoboda11 added a comment.

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.


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