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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 20 00:21:31 PST 2019


MaskRay added inline comments.


================
Comment at: llvm/test/tools/llvm-readobj/ELF/symbols.test:69
+# NO-T-ERR: Unknown command line argument '-t'
+## -s is a llvm-readobj option to dump sections.
+# RUN: llvm-readobj -s --elf-output-style=GNU %t64 | FileCheck %s --implicit-check-not="Symbol table"
----------------
jhenderson wrote:
> I //think// this should be "is an llvm-readobj".
(Slightly OT: I use "an R_*_IRELATIVE". I hope I used it correctly:) )


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

https://reviews.llvm.org/D71697





More information about the llvm-commits mailing list