[PATCH] D108120: [lld] Add CMake options to disable individual linkers

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 18 12:22:06 PDT 2021


sbc100 added a comment.

In D108120#2952982 <https://reviews.llvm.org/D108120#2952982>, @smeenai wrote:

> In D108120#2952966 <https://reviews.llvm.org/D108120#2952966>, @thakis wrote:
>
>> If this doesn't help you much upstream, and since it's not a big size change, is it really worth to add the complexity here upstream?
>>
>> (Don't worry about the gn build though, that's easy to update if this lands.)
>
> We're interested in this as well, and it'd be nice to have it upstream :) (I haven't run the size numbers on our end yet, to be fair.)

+1 to avoiding extra complexity unless the size wins really are significant (I'm not sure how we go about assessing that).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108120/new/

https://reviews.llvm.org/D108120



More information about the llvm-commits mailing list