[PATCH] D56780: [llvm-objdump] - Dump the archive headers when -all-headers is specified

[George Rimar via Phabricator via llvm-commits]

grimar marked an inline comment as done.
grimar added inline comments.


================
Comment at: test/tools/llvm-objdump/all-headers.test:17
+
+# RUN: yaml2obj %s > %t2
+# RUN: rm -f %t.a
----------------
jhenderson wrote:
> Not that it really matters, but I wonder if it would be better to simply copy %t?
I have no preference. Actually, we can even use the existent file twice, what do you think?
(it works, I just tested).



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

https://reviews.llvm.org/D56780