[PATCH] D72992: [llvm-objdump] - Add column headers for relocation printing

Mon Feb 3 03:55:42 PST 2020

jhenderson requested changes to this revision.
jhenderson added a comment.
This revision now requires changes to proceed.

Hi @liadz0rz,

I tried applying this locally and there were a number of test failures. Could you please make sure to run check-llvm to run all the tests again and fix any failures. In particular, the following tests all failed for me, with what look to be issues related to your patch:

  LLVM :: MC/ARM/dwarf-asm-multiple-sections-dwarf-2.s
  LLVM :: MC/ARM/dwarf-asm-multiple-sections.s
  LLVM :: MC/ARM/dwarf-asm-nonstandard-section.s
  LLVM :: MC/ARM/dwarf-asm-single-section.s
  LLVM :: MC/COFF/cfi-sections.s
  LLVM :: Object/objdump-relocations.test

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

https://reviews.llvm.org/D72992