[llvm-objcopy] Exclude empty sections in IHexWriter output
authorIan McIntyre <ianpmcintyre@gmail.com>
Sat, 12 Jun 2021 19:23:07 +0000 (12:23 -0700)
committerFangrui Song <i@maskray.me>
Sat, 12 Jun 2021 19:23:07 +0000 (12:23 -0700)
commit5899278758b66a5e72c1ae9695651c2aef7406c4
treea253429f754095b4b3a255eedf11cef53b8effe2
parentfae7debadcea335d4aaddee82406a8d10426e730
[llvm-objcopy] Exclude empty sections in IHexWriter output

IHexWriter was evaluating a section's physical address when deciding if
that section should be written to an output. This approach does not
account for a zero-sized section that has the same physical address as a
sized section. The behavior varies from GNU objcopy, and may result in a
HEX file that does not include all program sections.

The IHexWriter now excludes zero-sized sections when deciding what
should be written to the output. This affects the contents of the
writer's `Sections` collection; we will not try to insert multiple
sections that could have the same physical address. The behavior seems
consistent with GNU objcopy, which always excludes empty sections,
no matter the address.

The new test case evaluates the IHexWriter behavior when provided a
variety of empty sections that overlap or append a filled section. See
the input file's comments for more information. Given that test input,
and the change to the IHexWriter, GNU objcopy and llvm-objcopy produce
the same output.

Reviewed By: jhenderson, MaskRay, evgeny777

Differential Revision: https://reviews.llvm.org/D101332
llvm/test/tools/llvm-objcopy/ELF/Inputs/ihex-elf-sections2.yaml
llvm/test/tools/llvm-objcopy/ELF/ihex-writer-empty-sections.test [new file with mode: 0644]
llvm/tools/llvm-objcopy/ELF/Object.cpp