[PATCH] D46608: Add support for ThinLTO plugin option thinlto-object-suffix-replace

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 9 13:04:49 PDT 2018


ruiu added inline comments.


================
Comment at: lld/ELF/InputFiles.cpp:1030-1033
+  if (Config->ThinLTOIndexOnly) {
+    Path.consume_back(Config->ThinLTOObjectSuffixReplace.first.str());
+    UpdatedPath = Path.str() + Config->ThinLTOObjectSuffixReplace.second.str();
+  }
----------------
All tests including the new one you are adding in this patch seem to pass without these four lines of code. Is this needed?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D46608





More information about the llvm-commits mailing list