Page MenuHomePhabricator

[llvm-objcopy][ELF] Allow --dump-section to dump an empty non-SHT_NOBITS section
ClosedPublic

Authored by MaskRay on May 4 2020, 10:12 AM.

Diff Detail

Event Timeline

MaskRay created this revision.May 4 2020, 10:12 AM
jhenderson accepted this revision.May 5 2020, 12:21 AM

LGTM.

llvm/test/tools/llvm-objcopy/ELF/dump-section.test
51

Nit: the prevailing pattern for FileCheck directives in this file is currently #CHECK: (no space). I'm okay if you want to change the other lines to match the # CHECK: style, but perhaps do as a separate NFC commit. Alternatively, make this match the existing ones.

This revision is now accepted and ready to land.May 5 2020, 12:21 AM
MaskRay updated this revision to Diff 262120.May 5 2020, 8:24 AM
MaskRay marked an inline comment as done.

#CHECK -> # CHECK in a separate commit