[lld] r316547 - COFF: Don't add /manifest* flags to the response file.

[Peter Collingbourne via llvm-commits]

Author: pcc
Date: Tue Oct 24 22:00:54 2017
New Revision: 316547

URL: http://llvm.org/viewvc/llvm-project?rev=316547&view=rev
Log:
COFF: Don't add /manifest* flags to the response file.

If /manifest:embed is enabled we're already creating a resource file
out of these flags and adding it to the linkrepro, and it doesn't
seem worth being able to repro side-by-side manifests.

Includes a test that covers this commit as well as r315948.

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

Added:
    lld/trunk/test/COFF/linkrepro-manifest.test
Modified:
    lld/trunk/COFF/Driver.cpp

Modified: lld/trunk/COFF/Driver.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/COFF/Driver.cpp?rev=316547&r1=316546&r2=316547&view=diff
==============================================================================
--- lld/trunk/COFF/Driver.cpp (original)
+++ lld/trunk/COFF/Driver.cpp Tue Oct 24 22:00:54 2017
@@ -410,6 +410,12 @@ static std::string createResponseFile(co
     case OPT_INPUT:
     case OPT_defaultlib:
     case OPT_libpath:
+    case OPT_manifest:
+    case OPT_manifest_colon:
+    case OPT_manifestdependency:
+    case OPT_manifestfile:
+    case OPT_manifestinput:
+    case OPT_manifestuac:
       break;
     default:
       OS << toString(Arg) << "\n";

Added: lld/trunk/test/COFF/linkrepro-manifest.test
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/COFF/linkrepro-manifest.test?rev=316547&view=auto
==============================================================================
--- lld/trunk/test/COFF/linkrepro-manifest.test (added)
+++ lld/trunk/test/COFF/linkrepro-manifest.test Tue Oct 24 22:00:54 2017
@@ -0,0 +1,12 @@
+REQUIRES: x86, gnutar, manifest_tool
+
+RUN: rm -rf %t && mkdir %t
+RUN: lld-link -entry:__ImageBase -nodefaultlib -linkrepro:%t -manifest:embed %p/Inputs/std32.lib -subsystem:console
+RUN: tar tf %t/repro.tar | FileCheck --check-prefix=LIST %s
+RUN: tar xOf %t/repro.tar repro/response.txt | FileCheck %s
+
+LIST: manifest.res
+
+CHECK-NOT: -manifest
+CHECK: .manifest.res
+CHECK-NOT: -manifest