[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 15:01:19 PDT 2016


pcc added inline comments.


================
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)
----------------
I think this would result in a default parallelism level of 1. Can we avoid passing a ThinBackend if the user did not specify a parallelism level?


https://reviews.llvm.org/D25452





More information about the llvm-commits mailing list