[PATCH] D76907: [PPCInstPrinter] Print conditional branches as `bt 2, $target` instead of `bt 2, .+$imm`

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 31 14:53:25 PDT 2020


sfertile accepted this revision.
sfertile added a comment.
This revision is now accepted and ready to land.

1 comment, but otherwise LGTM.



================
Comment at: llvm/lib/Target/PowerPC/Disassembler/PPCDisassembler.cpp:63
 
+static DecodeStatus decodeCondBrOperand(MCInst &Inst, unsigned Imm,
+                                        uint64_t /*Address*/,
----------------
Minor nit: Maybe `decodeCondBrTarget` since condition branches have other operands.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76907





More information about the llvm-commits mailing list