[PATCH] D71697: [llvm-readobj] - Merge `gnu-symbols.test` to `symbols.test` and cleanup.

[Fangrui Song via Phabricator via llvm-commits]

MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/test/tools/llvm-readobj/ELF/symbols.test:83
+
+## Case 4: Test that `--symbols` triggers `--dyn-symbols` implicitly for llvm-readelf,
+##         but not for llvm-readobj.
----------------
Rather than "trigger ... implicitly", you may say --symbols implies --dyn-symbols for llvm-readelf.


================
Comment at: llvm/test/tools/llvm-readobj/ELF/symbols.test:89
+## Case 5: Test that `--dyn-symbols` does not trigger `--symbols` implicitly.
+# RUN: llvm-readobj --dyn-symbols %t64 | FileCheck %s --implicit-check-not="{{^}}Symbols ["
+# RUN: llvm-readelf --dyn-symbols %t64 | FileCheck %s --implicit-check-not="Symbol table '.symtab'"
----------------
`{{^}}` can be deleted.

 The fact that `DynamicSymbols [` does not follow a `{{^}}` made me wonder why `{{^}}` is not here...


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

https://reviews.llvm.org/D71697