[PATCH] D73698: [ThinLTO] Import globals recursively

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 30 11:35:17 PST 2020


tejohnson added a comment.

In D73698#1850176 <https://reviews.llvm.org/D73698#1850176>, @evgeny777 wrote:

> > FYI we had to disable a related variable importing patch due to some widespread order of magnitude increases in thin link time (see D73724 <https://reviews.llvm.org/D73724>). Until that is addressed, this should go under an option off by default, or just held off.
>
> I see. Is it possible to reproduce this on some publicly available code base? Like chromium for example


Not sure, I don't know offhand how to build that one. Going to see if I can root cause where the huge increases come from exactly, will get back.


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

https://reviews.llvm.org/D73698





More information about the llvm-commits mailing list