[PATCH] D63847: [MC] Add MCInstrAnalysis::evaluateMemoryOperandAddress

Mon Jul 1 00:55:36 PDT 2019

seiya added a comment.

In D63847#1561882 <https://reviews.llvm.org/D63847#1561882>, @MaskRay wrote:

> Are you planning to use it in llvm-objdump to symbolize instructions like: `mov    0x206a3d(%rip),%rax        # 207fe8 <__cxa_finalize at plt+0x2066a8>`?
>
> When you make a dependent llvm-objdump change, you may add the test somewhere under test/tools/llvm-objdump

Yes. The example you showed is exactly what I'm going to implement. I plan to submit a separate patch which add tests along with the implementation of the feature in llvm-objdump.

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

https://reviews.llvm.org/D63847