[PATCH] D63013: [llvm-dwarfdump] Add -o to help text and remove --out-file from documentation

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

In D63013#1537125 <https://reviews.llvm.org/D63013#1537125>, @aprantl wrote:

> In D63013#1536440 <https://reviews.llvm.org/D63013#1536440>, @jhenderson wrote:
>
> > In D63013#1535260 <https://reviews.llvm.org/D63013#1535260>, @aprantl wrote:
> >
> > > I'm mildly against this change. IIRC, the idea was that --out-file is a long-form option that nobody uses in practice and we wanted people to prefer -o instead, which is what people are most familiar with working with  compilers. So if -o shows up in the --help output, I'd leave it at that.
> >
> >
> > Could we just delete --out-file, if we don't want people to use it?
>
>
> Not any more, I'm afraid :-)
>  Once an option is out there we will no doubt get complaints that we broke somebody's scripts if we remove it.


(I'd be in favor of removing it, if it doesn't serve much purpose - does anyone know the history of why it was added?)

We've changed the behavior of llvm-dwarfdump pretty significantly over the years (these days it prints with coloured output, only dumping debug_info by default, and in non-verbose mode (skipping forms and other things)) - it's a fairly "compiler/debugger developer centric tool" which is a small enough audience we've been pretty comfortable making usability improvements even if they'd break some kind of compatibility.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63013