[PATCH] D72093: [ELF][Hexagon][test] Use llvm-readobj to check relocations. NFC

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 2 11:05:34 PST 2020


MaskRay added a comment.

In D72093#1801658 <https://reviews.llvm.org/D72093#1801658>, @sidneym wrote:

> This is fine with me.  This patch doesn't remove the objdump checks are both necessary?


llvm-readobj -r checks input relocations. llvm-objdump -d checks output disassembly. Both are necessary.

I usually prefer llvm-readelf for other checks, like -S (section table), -s (symbol table) as its output is concise. For -r, the llvm-readobj output looks better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72093





More information about the llvm-commits mailing list