[PATCH] D84191: [llvm-objdump] Symbolize binary addresses for low-noisy asm diff.

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 11 22:37:29 PDT 2020


hoyFB marked 2 inline comments as done.
hoyFB added a comment.

In D84191#2212199 <https://reviews.llvm.org/D84191#2212199>, @MaskRay wrote:

> The last update looks good. Let's wait a bit for binutils' comment.. I also have a question about whether we should name the option `--symbolize-operands`, or is there a better name. It drops the existing offset/address operand as well as adding a symbol.

Printing a symbol instead of an offset/address is sort of symbolization to me. That said, a more meaningful name is welcomed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84191



More information about the llvm-commits mailing list