[PATCH] D29526: Add an option to use the MSVC linker to link LTO-generated object files.

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added inline comments.


================
Comment at: lld/COFF/DriverUtils.cpp:647
+  Executor E("link.exe");
+  E.add("/nologo");
+  for (TemporaryFile &T : Temps)
----------------
Should this create a response file instead of passing the arguments directly?


================
Comment at: lld/COFF/DriverUtils.cpp:652
+    switch (Arg->getOption().getID()) {
+    case OPT_INPUT:
+    case OPT_defaultlib:
----------------
Should this strip all inputs or just bitcode files?


================
Comment at: lld/COFF/DriverUtils.cpp:658
+    case OPT_msvclto:
+      // LLD-specific options are stripped.
+      break;
----------------
What about `/opt:lld*`?


================
Comment at: lld/COFF/SymbolTable.cpp:354
 
-void SymbolTable::addCombinedLTOObjects() {
-  if (BitcodeFiles.empty())
-    return;
-
+std::vector<StringRef> SymbolTable::compile() {
   LTO.reset(new BitcodeCompiler);
----------------
Maybe call this something like `compileBitcodeFiles`?


https://reviews.llvm.org/D29526