[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