[PATCH] D75756: [llvm-objdump] Teach `llvm-objdump` dump dynamic symbols.

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: llvm/test/tools/llvm-objdump/unimplemented-features.test:14
+# XCOFF object file.
+# RUN: cp %p/Inputs/xcoff-section-headers.o %t3.o
+# RUN: llvm-objdump --dynamic-syms %t3.o 2>&1 | \
----------------
Higuoxing wrote:
> jhenderson wrote:
> > No need for the `cp`. It's okay to reference the binary directly in the llvm-objdump call here.
> I was using `cp` to shorten the file name as `%t3`, which is intended to replace `[[FILE]]`. Is this acceptable?
Personally, I don't think that really gains us anything.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75756