[PATCH] D31986: [coff] default to multiple parallel ThinLTO jobs
Hans Wennborg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 12 11:33:59 PDT 2017
hans accepted this revision.
hans added a comment.
In https://reviews.llvm.org/D31986#725147, @pcc wrote:
> In https://reviews.llvm.org/D31986#725140, @hans wrote:
>
> > Seems reasonable to me, but I'm not familiar with the difference between LTOJobs and LTOPartitions. Does it make sense to have multiple jobs but just one partition? I thought the point of ThinLTO was to process multiple partitions in parallel.
>
>
> LTOPartitions is used by parallel LTO code generation, which is a separate feature to ThinLTO.
I see. LGTM then.
https://reviews.llvm.org/D31986
More information about the llvm-commits
mailing list