[PATCH] D25452: [LTO] Split the options for ThinLTO jobs and Regular LTO partitions
Peter Collingbourne via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 10 16:13:19 PDT 2016
pcc accepted this revision.
pcc added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: ELF/Driver.cpp:487
+ error("--lto-partitions: number of threads must be > 0");
+ Config->ThinLtoJobs = getInteger(Args, OPT_thinlto_jobs, -1);
+ if (Config->ThinLtoJobs == 0)
----------------
Maybe `-1u` here and below?
https://reviews.llvm.org/D25452
More information about the llvm-commits
mailing list