[PATCH] D77310: [ms] Add new /PDBSTREAM option to lld-link allowing injection of streams into PDB files.

Eric Astor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 6 09:44:45 PDT 2020


epastor added inline comments.


================
Comment at: lld/COFF/Driver.cpp:1281
+        const std::string file = nameFile.second.str();
+        config->namedStreams[name] = file;
+      }
----------------
thakis wrote:
> Should this do anything if you try to overwrite an existing stream? (Like, error?)
I've mimicked other command line flags; the last one with the same name will take precedence.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77310/new/

https://reviews.llvm.org/D77310





More information about the llvm-commits mailing list