[PATCH] D86283: [llvm-readelf][test] - Refine the merged.test

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 20 06:17:12 PDT 2020


jhenderson added inline comments.


================
Comment at: llvm/test/tools/llvm-readobj/ELF/merged.test:10-14
+## Check that the output without -a (--all) option is the same. We do it
+## to verify that each of the options is handled, and not just the first
+## one nor just -a, which triggers all other options.
+# RUN: llvm-readelf -eWhSrnudlVgIs %t.o > %t.merged.no.a
+# RUN: cmp %t.merged %t.merged.no.a
----------------
We probably want similar testing for -e, since that is the same as specifying -hlS, I believe.


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

https://reviews.llvm.org/D86283



More information about the llvm-commits mailing list