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

Xing GUO via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 24 03:12:17 PDT 2020


Higuoxing marked an inline comment as done.
Higuoxing 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 | \
----------------
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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75756





More information about the llvm-commits mailing list