[PATCH] D18999: [ELF/LTO] Parallel Codegen for LLD

[Peter Collingbourne via llvm-commits]

pcc added inline comments.

================
Comment at: ELF/Options.td:237-238
@@ -237,1 +236,4 @@
+// LTO-related options.
+def jobs : Joined<["--"], "jobs=">,
+  HelpText<"Number of threads to run codegen">;
 def disable_verify : Flag<["-"], "disable-verify">;
----------------
ruiu wrote:
> I prefer --thread-count because gold has that option.
Gold's `--thread-count` does not affect LTO. Furthermore, parallel LTO codegen can result in the linker producing different binaries based on the number of threads, and users of `--thread-count` probably wouldn't expect that option to change the binary. I think this should probably be something prefixed with `--lto-`.


http://reviews.llvm.org/D18999