[PATCH] D72402: [build][NFC] limit parallel link to reduce memory usage

Kókai Péter via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 9 10:05:06 PST 2020


Kokan added a comment.

In D72402#1810269 <https://reviews.llvm.org/D72402#1810269>, @dblaikie wrote:

> Could we do this the other way around - and "AND LINKER_IS_GNULD"? Because so far as I know gold has reasonable memory usage too (& LD64), etc. I /think/ it's just GNULD that's the outlier here. (I can't speak for SOLARISLD, admittedly)


This looks reasonable for me. The part that creates `LINKER_IS_GNULD` is implemented in `include(AddLLVM)`, that is included much later then this `HandleLLVMOptions`. I will look into if that can be changed.


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

https://reviews.llvm.org/D72402





More information about the llvm-commits mailing list