[PATCH] D64706: [tools] [llvm-objdump] Default to reading from stdin not a.out

Mon Jul 15 09:53:30 PDT 2019

jhenderson added a comment.

In D64706#1585753 <https://reviews.llvm.org/D64706#1585753>, @compnerd wrote:

> `objcopy` is often used in autotools to run tests of what the compiler is doing.  I'm afraid that changing that behaviour will break the autoconf tests which rely on the behaviour (`a.out` is what the compiler generates by default, and that is why the tool defaults to that filename).  So, if you have something like the following:
>
>   cc test.c
>   objcopy --extract-dwo
>
>
> to test whether you can do split-debug with dwarf packages, it will break.

It's worth noting that this is not about llvm-objcopy. It's about llvm-objdump. I don't believe our llvm-objcopy has the behaviour you've described implemented anyway. That being said, I think it's a valid point and you should bring it up on the thread on the mailing list to discuss it, so that this discussion gets more visibility: https://lists.llvm.org/pipermail/llvm-dev/2019-July/133642.html

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

https://reviews.llvm.org/D64706