[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