Revert r371729: lld-link: Make /linkrepro: take a filename, not a directory.
This reverts commit r371729 because /linkrepro option also exists
in Microsoft link.exe and their linker takes not a filename but a
directory name as an argument for /linkrepro.
Differential Revision: https://reviews.llvm.org/D68378
git-svn-id: https://llvm.org/svn/llvm-project/lld/trunk@373703 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/COFF/Driver.cpp b/COFF/Driver.cpp
index 7688ff3..24814cc 100644
--- a/COFF/Driver.cpp
+++ b/COFF/Driver.cpp
@@ -51,7 +51,7 @@
using namespace llvm;
using namespace llvm::object;
using namespace llvm::COFF;
-using namespace llvm::sys;
+using llvm::sys::Process;
namespace lld {
namespace coff {
@@ -1134,15 +1134,17 @@
config->mingw = args.hasArg(OPT_lldmingw);
if (auto *arg = args.getLastArg(OPT_linkrepro)) {
- const char *path = arg->getValue();
+ SmallString<64> path = StringRef(arg->getValue());
+ sys::path::append(path, "repro.tar");
Expected<std::unique_ptr<TarWriter>> errOrWriter =
- TarWriter::create(path, path::stem(path));
+ TarWriter::create(path, "repro");
+
if (errOrWriter) {
tar = std::move(*errOrWriter);
- tar->append("version.txt", getLLDVersion() + "\n");
} else {
- error("/linkrepro: " + toString(errOrWriter.takeError()));
+ error("/linkrepro: failed to open " + path + ": " +
+ toString(errOrWriter.takeError()));
}
}
diff --git a/docs/ReleaseNotes.rst b/docs/ReleaseNotes.rst
index 9c91a64..8cc7065 100644
--- a/docs/ReleaseNotes.rst
+++ b/docs/ReleaseNotes.rst
@@ -29,11 +29,6 @@
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.
-* The new `/lldignoreenv` flag makes lld-link ignore environment variables
- like `%LIB%`.
* ...
MinGW Improvements
diff --git a/test/COFF/linkrepro-pdb.test b/test/COFF/linkrepro-pdb.test
index 3cd7403..f248c43 100644
--- a/test/COFF/linkrepro-pdb.test
+++ b/test/COFF/linkrepro-pdb.test
@@ -4,7 +4,7 @@
RUN: yaml2obj %S/Inputs/pdb-type-server-simple-a.yaml -o a.obj
RUN: yaml2obj %S/Inputs/pdb-type-server-simple-b.yaml -o b.obj
RUN: llvm-pdbutil yaml2pdb %S/Inputs/pdb-type-server-simple-ts.yaml -pdb ts.pdb
-RUN: lld-link a.obj b.obj -entry:main -debug -out:%t.exe -pdb:%t.pdb -nodefaultlib -linkrepro:repro.tar
+RUN: lld-link a.obj b.obj -entry:main -debug -out:%t.exe -pdb:%t.pdb -nodefaultlib -linkrepro:.
RUN: tar xOf repro.tar repro/%:t/ts.pdb > repro-ts.pdb
RUN: diff ts.pdb repro-ts.pdb
@@ -15,7 +15,7 @@
PDB-NEXT: -pdb:linkrepro-pdb.test.tmp.pdb
RUN: yaml2obj < %p/Inputs/export.yaml > %t1.obj
-RUN: lld-link /out:%t1.dll /dll %t1.obj /implib:%t1.lib /export:exportfn1 /export:exportfn2 /linkrepro:repro.tar
+RUN: lld-link /out:%t1.dll /dll %t1.obj /implib:%t1.lib /export:exportfn1 /export:exportfn2 /linkrepro:.
RUN: tar xf repro.tar
RUN: cat repro/response.txt | FileCheck -check-prefix=IMP %s
diff --git a/test/COFF/linkrepro-res.test b/test/COFF/linkrepro-res.test
index aac614f..cf0aa16 100644
--- a/test/COFF/linkrepro-res.test
+++ b/test/COFF/linkrepro-res.test
@@ -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:repro.tar /out:%t.exe
+# RUN: /entry:__ImageBase /linkrepro:. /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
diff --git a/test/COFF/linkrepro.test b/test/COFF/linkrepro.test
index 4bb5e59..8f42234 100644
--- a/test/COFF/linkrepro.test
+++ b/test/COFF/linkrepro.test
@@ -6,7 +6,7 @@
# RUN: cd %t.dir/build1
# RUN: lld-link %t.obj %p/Inputs/std32.lib /subsystem:console \
-# RUN: /entry:main@0 /linkrepro:repro.tar /out:%t.exe
+# RUN: /entry:main@0 /linkrepro:. /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@0 /linkrepro:repro.tar /out:%t.exe
+# RUN: /entry:main@0 /linkrepro:. /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@0 /linkrepro:repro.tar /out:%t.exe
+# RUN: /entry:main@0 /linkrepro:. /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