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

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 3 13:34:02 PDT 2020


thakis added a comment.

Seems reasonable. Can you add a test?



================
Comment at: lld/COFF/Driver.cpp:1281
+        const std::string file = nameFile.second.str();
+        config->namedStreams[name] = file;
+      }
----------------
Should this do anything if you try to overwrite an existing stream? (Like, error?)


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