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

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

In D59530#1434691 <https://reviews.llvm.org/D59530#1434691>, @aganea wrote:

> In D59530#1434688 <https://reviews.llvm.org/D59530#1434688>, @mstorsjo wrote:
>
> > 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?).
>
>
> Yes, however our build system generates cmd-lines with full or relative paths for these options, which makes the response unusable, unless you edit it (because the paths aren't there on the reproducer's machine)
>
> Are you suggesting that we strip those options from their path, and only kept the filename? That would be a good alternative.


That sounds like a good alternative to me, yes.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D59530