COFF: Don't add /manifest* flags to the response file.

[pcc]

Summary

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.

No tests because this requires Windows and the linkrepro tests
require not-Windows.

[ruiu]

LGTM

lld/trunk/COFF/Driver.cpp

static std::string createResponseFile(const opt::InputArgList &Args,
   raw_svector_ostream OS(Data);
 
   for (auto *Arg : Args) {
     switch (Arg->getOption().getID()) {
     case OPT_linkrepro:
     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";
     }
   }
 
   for (StringRef Path : SearchPaths) {
     std::string RelPath = relativeToRoot(Path);

lld/trunk/test/COFF/linkrepro-manifest.test

+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