[PATCH] D59530: [LLD][COFF] Fix /linkrepro with options that take a filename or path

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 19 05:54:27 PDT 2019


mstorsjo added a comment.

Aren't all of `/implib`, `/pdb`and `/out` output files? In that sense it doesn't matter where they are written in the repro case (and writing them locally instead of somewhere else with an absolute path probably is more convenient?).

The one that would matter though, which iirc isn't handled right now, is when def files are passed as input. IIRC they aren't copied into the repro bundle.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D59530





More information about the llvm-commits mailing list