[PATCH] D76591: [PPCInstPrinter] Change printBranchOperand(calltarget) to print the target address in hexadecimal form

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 24 09:39:28 PDT 2020


MaskRay added a comment.

In D76591#1939318 <https://reviews.llvm.org/D76591#1939318>, @jasonliu wrote:

> @nemanjai 
>  AIX system assembler does not seem to like this new form.
>
>   Assembler:
>   test.s: line 46: 1252-086 The target of the branch instruction
>           must be a relocatable or external expression.
>


llvm-objdump -d output may not be assembled back. That said, I think most users like the PC relative immediate to be displayed as the target address. If AIX users don't agree, I can special case it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76591





More information about the llvm-commits mailing list