[PATCH] D47799: [COFF] add /errorrepro to save reproducer on error

[Bob Haarman via Phabricator via llvm-commits]

inglorion added a comment.

> I still don't see a compelling reason to do this to be honest.

Fair enough, and if the consensus is that we'd rather not have this, I'll withdraw it. You've reduced the problem that originally prompted me to write this (and you're right it doesn't capture the object files LTO generates). However, aside from that specific problem, I still think that something like this would be useful to diagnose linker errors we may encounter in the future, particularly ones that don't reproduce reliably. As I pointed out earlier, /linkrepro isn't really what I would want for that because you either need to re-run the link until you hit the same failure again, which could be very time consuming, or pre-emptively run with /linkrepro, which slows down links and writes lots of bytes you usually don't care about.


https://reviews.llvm.org/D47799