[PATCH] D133110: lld: Include name of output file in "failed to write output" diag

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 07:09:36 PDT 2022


thakis added a comment.

I see this fairly often on Windows, where you can't write to a file that another process – such as a virus scanner – has open: http://45.33.8.238/win/65440/step_4.txt I think it'd be nice to include the file name in the diag.

Come to think of it, maybe we should retry the write after it fails? Might help on Windows. On the other hand, that just hides the problem and doesn't fully fix it, so not sure that's better.


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

https://reviews.llvm.org/D133110



More information about the llvm-commits mailing list