[PATCH] D46034: Support for distributed ThinLTO options

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 2 13:38:14 PDT 2018


pcc accepted this revision.
pcc added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lld/ELF/LTO.cpp:259
+  // If LazyObjFile has not been added to link, emit empty index files
+  if (Config->ThinLTOIndexOnly) {
+    for (LazyObjFile *F : LazyObjFiles)
----------------
Remove this one as well.


================
Comment at: lld/ELF/Options.td:428-431
+def thinlto_index_only: J<"thinlto-index-only=">,
+  HelpText<"Write individual backend index files for LTO ">;
+def thinlto_prefix_replace : F<"thinlto-prefix-replace">,
+  HelpText<"Controls location of distributed backend files">;
----------------
These lines aren't needed, we only support the new flags as `plugin-opt`s.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D46034





More information about the llvm-commits mailing list