[PATCH] D48810: [llvm-objdump] Add --file-headers (-f) option

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: test/tools/llvm-objdump/file-headers-coff.test:2
+# RUN: yaml2obj %s > %t
+# RUN: llvm-objdump -f %t | FileCheck %s
+
----------------
One more thing: since --file-headers is an alias of -f, I'd like to see both tested, to show they do the same thing. It should just be a case of duplicating this line in each of the tests, but with the other version of the switch (the same goes with other similar changes, such as D48904).


Repository:
  rL LLVM

https://reviews.llvm.org/D48810