[PATCH] D71453: [llvm-objdump] Print target address instead of call/j/br pc-rel immediate

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

This is basically the largest blocker that prevents me from using llvm-objdump regularly / or why I am still using `powerpc-linux-gnu-objdump -d`, `objdump -d`, etc... I thought about fixing this, but it would require a lot of test update.

If `lld` is in your `-DLLVM_ENABLE_PROJECTS=` list, try `ninja check-lld`, there should be lots of failures. There is an `llvm-objdump` option `--print-imm-hex` which is also used to decide whether an immediate should be printed in the hexadecimal form, but the target InstrPrinter code may not consistently respect it. (I've updated many lld tests to use `--print-imm-hex`, but I believe there are still lots of remaining issues... After `--print-imm-hex` becomes the default, we should probably do another clean-up to remove the option)


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

https://reviews.llvm.org/D71453