[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