[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