[PATCH] D65645: [llvm-dlltool] Remove support for implying output name

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

For the record, while this might look like a surprising patch (removing functionality, why?), I noticed this while looking into the dlltool option `-y` (for generating a delayload import library). Dlltool has the options `-l` for specifying the output name of a normal import library, and the option `-y` for specifying the output name of a delayload import library. If using the `-y` option only, the current behaviour of implicitly picking an output name and writing to it, if no `-l` option was specified, seemed odd, and I noticed that GNU dlltool didn't actually have that behaviour.

(I don't have anything upstream-worthy regarding actually implementing the `-y` option in dlltool; the sole user so far, wine, managed to work around it and use the lld delayload option instead.)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65645