[PATCH] D59823: [llvm-readelf]Merge dynamic and static relocation printing to avoid code duplication

Tue Mar 26 08:54:50 PDT 2019

jhenderson created this revision.
jhenderson added reviewers: grimar, rupprecht, Higuoxing.
Herald added subscribers: kristof.beyls, javed.absar.
Herald added a project: LLVM.

The majority of the printRelocation and printDynamicRelocation functions were identical. This patch factors this all out into a new function. There are a couple of minor differences to do with printing of symbols without names, but I think these are harmless, and in some cases a small improvement.

Repository:
  rL LLVM

https://reviews.llvm.org/D59823

Files:
  test/tools/llvm-readobj/elf-reloc-zero-name-or-value.test
  tools/llvm-readobj/ELFDumper.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59823.192285.patch
Type: text/x-patch
Size: 7821 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190326/73d68535/attachment.bin>