[PATCH] D102603: [llvm-objdump] Print the DEBUG type under `--section-headers`.

EsmeYi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 24 23:25:09 PDT 2021


Esme added inline comments.


================
Comment at: llvm/test/tools/llvm-objdump/XCOFF/section-headers.test:1-5
-# RUN: llvm-objdump --section-headers %p/Inputs/xcoff-section-headers.o | \
-# RUN: FileCheck %s
-
-# RUN: llvm-objdump --section-headers %p/Inputs/xcoff-long-sec-names.o | \
-# RUN: FileCheck --check-prefix=LONG %s
----------------
Higuoxing wrote:
> It looks that `xcoff-section-headers.o` and `xcoff-long-sec-names.o` are useless after this change. Can we remove them?
`xcoff-long-sec-names.o` is useless, but `xcoff-section-headers.o` is used by other tests. Those tests may be replaced by yaml-docs in the future, which should be another work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102603



More information about the llvm-commits mailing list