[lld] r371729 - lld-link: Make /linkrepro: take a filename, not a directory.

[Nico Weber via llvm-commits]

Author: nico
Date: Thu Sep 12 04:44:13 2019
New Revision: 371729

URL: http://llvm.org/viewvc/llvm-project?rev=371729&view=rev
Log:
lld-link: Make /linkrepro: take a filename, not a directory.

This makes lld-link behave like ld.lld. I don't see a reason for
the two drivers to have different behavior here.

While here, also make lld-link add a version.txt to the tar, like
ld.lld does.

Differential Revision: https://reviews.llvm.org/D67461

Modified:
    lld/trunk/COFF/Driver.cpp
    lld/trunk/docs/ReleaseNotes.rst
    lld/trunk/test/COFF/linkrepro-res.test
    lld/trunk/test/COFF/linkrepro.test

Modified: lld/trunk/COFF/Driver.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/COFF/Driver.cpp?rev=371729&r1=371728&r2=371729&view=diff
==============================================================================
--- lld/trunk/COFF/Driver.cpp (original)
+++ lld/trunk/COFF/Driver.cpp Thu Sep 12 04:44:13 2019
@@ -50,7 +50,7 @@
 using namespace llvm;
 using namespace llvm::object;
 using namespace llvm::COFF;
-using llvm::sys::Process;
+using namespace llvm::sys;
 
 namespace lld {
 namespace coff {
@@ -1138,17 +1138,15 @@ void LinkerDriver::link(ArrayRef<const c
   config->mingw = args.hasArg(OPT_lldmingw);
 
   if (auto *arg = args.getLastArg(OPT_linkrepro)) {
-    SmallString<64> path = StringRef(arg->getValue());
-    sys::path::append(path, "repro.tar");
+    const char *path = arg->getValue();
 
     Expected<std::unique_ptr<TarWriter>> errOrWriter =
-        TarWriter::create(path, "repro");
-
+        TarWriter::create(path, path::stem(path));
     if (errOrWriter) {
       tar = std::move(*errOrWriter);
+      tar->append("version.txt", getLLDVersion() + "\n");
     } else {
-      error("/linkrepro: failed to open " + path + ": " +
-            toString(errOrWriter.takeError()));
+      error("/linkrepro: " + toString(errOrWriter.takeError()));
     }
   }
 

Modified: lld/trunk/docs/ReleaseNotes.rst
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/docs/ReleaseNotes.rst?rev=371729&r1=371728&r2=371729&view=diff
==============================================================================
--- lld/trunk/docs/ReleaseNotes.rst (original)
+++ lld/trunk/docs/ReleaseNotes.rst Thu Sep 12 04:44:13 2019
@@ -29,6 +29,9 @@ ELF Improvements
 COFF Improvements
 -----------------
 
+* /linkrepro: now takes the filename of the tar archive it writes, instead
+  of the name of a directory that a file called "repro.tar" is created in,
+  matching the behavior of ELF lld.
 * ...
 
 MinGW Improvements

Modified: lld/trunk/test/COFF/linkrepro-res.test
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/COFF/linkrepro-res.test?rev=371729&r1=371728&r2=371729&view=diff
==============================================================================
--- lld/trunk/test/COFF/linkrepro-res.test (original)
+++ lld/trunk/test/COFF/linkrepro-res.test Thu Sep 12 04:44:13 2019
@@ -4,7 +4,7 @@
 # RUN: mkdir -p %t.dir/build
 # RUN: cd %t.dir/build
 # RUN: lld-link %p/Inputs/resource.res /subsystem:console /machine:x64 \
-# RUN:   /entry:__ImageBase /linkrepro:. /out:%t.exe
+# RUN:   /entry:__ImageBase /linkrepro:repro.tar /out:%t.exe
 # RUN: tar xf repro.tar
 # RUN: diff %p/Inputs/resource.res repro/%:p/Inputs/resource.res
 # RUN: FileCheck %s --check-prefix=RSP < repro/response.txt

Modified: lld/trunk/test/COFF/linkrepro.test
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/COFF/linkrepro.test?rev=371729&r1=371728&r2=371729&view=diff
==============================================================================
--- lld/trunk/test/COFF/linkrepro.test (original)
+++ lld/trunk/test/COFF/linkrepro.test Thu Sep 12 04:44:13 2019
@@ -6,7 +6,7 @@
 
 # RUN: cd %t.dir/build1
 # RUN: lld-link %t.obj %p/Inputs/std32.lib /subsystem:console \
-# RUN:   /entry:main at 0 /linkrepro:. /out:%t.exe
+# RUN:   /entry:main at 0 /linkrepro:repro.tar /out:%t.exe
 # RUN: tar xf repro.tar
 # RUN: diff %t.obj repro/%:t.obj
 # RUN: diff %p/Inputs/std32.lib repro/%:p/Inputs/std32.lib
@@ -14,7 +14,7 @@
 
 # RUN: cd %t.dir/build2
 # RUN: lld-link %t.obj /libpath:%p/Inputs /defaultlib:std32 /subsystem:console \
-# RUN:   /entry:main at 0 /linkrepro:. /out:%t.exe
+# RUN:   /entry:main at 0 /linkrepro:repro.tar /out:%t.exe
 # RUN: tar xf repro.tar
 # RUN: diff %t.obj repro/%:t.obj
 # RUN: diff %p/Inputs/std32.lib repro/%:p/Inputs/std32.lib
@@ -22,7 +22,7 @@
 
 # RUN: cd %t.dir/build3
 # RUN: env LIB=%p/Inputs lld-link %t.obj /defaultlib:std32 /subsystem:console \
-# RUN:   /entry:main at 0 /linkrepro:. /out:%t.exe
+# RUN:   /entry:main at 0 /linkrepro:repro.tar /out:%t.exe
 # RUN: tar xf repro.tar
 # RUN: diff %t.obj repro/%:t.obj
 # RUN: diff %p/Inputs/std32.lib repro/%:p/Inputs/std32.lib