[PATCH] D75153: [ThinLTO] Allow usage of all SMT threads in the system

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 4 13:30:17 PST 2020


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

I'm busy and I haven't looked at the code in detail, but I'm OK with going back to the old way of doing things. I think honoring user requests to use more threads than cores is an important use case. We had plans at one point to add indirection points to allow us to distribute these backend actions to other machines. Essentially, this would be done by having LLD invoke `$mydiscc clang -c foo.o -fuse-thin-lto-index=foo.thinlto.bc`. We've gone a different direction, so that change is not likely to come soon, but it seems like a reasonable use case where one would want to pass -j LARGE and have it be honored.


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

https://reviews.llvm.org/D75153





More information about the llvm-commits mailing list