diff options
author | Lioncash <mathew1800@gmail.com> | 2015-02-27 21:57:38 -0500 |
---|---|---|
committer | Lioncash <mathew1800@gmail.com> | 2015-02-27 21:59:30 -0500 |
commit | 9ed34889258cd3da1c46696cad5e96011c3a5a0b (patch) | |
tree | 7a5c781fcfa6e98d67b0f7244b58c338190fc650 /src/core/arm/disassembler | |
parent | c9ef377afaa038797de6c08da9f204acf67ed1fc (diff) |
arm_disasm: Show conditional code for BKPT instructions.
Changed cond_to_str to take a uint32, since unsigned numbers are only ever passed to it, and this can be a source of warnings for some compilers (also indexing an array without bounds checking a signed number is kind of iffy).
Diffstat (limited to 'src/core/arm/disassembler')
-rw-r--r-- | src/core/arm/disassembler/arm_disasm.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/core/arm/disassembler/arm_disasm.cpp b/src/core/arm/disassembler/arm_disasm.cpp index f7c7451e..5a8a113c 100644 --- a/src/core/arm/disassembler/arm_disasm.cpp +++ b/src/core/arm/disassembler/arm_disasm.cpp @@ -131,7 +131,7 @@ static const char *shift_names[] = { "ROR" }; -static const char* cond_to_str(int cond) { +static const char* cond_to_str(uint32_t cond) { return cond_names[cond]; } @@ -337,8 +337,9 @@ std::string ARM_Disasm::DisassembleBX(uint32_t insn) std::string ARM_Disasm::DisassembleBKPT(uint32_t insn) { + uint8_t cond = (insn >> 28) & 0xf; uint32_t immed = (((insn >> 8) & 0xfff) << 4) | (insn & 0xf); - return Common::StringFromFormat("bkpt\t#%d", immed); + return Common::StringFromFormat("bkpt%s\t#%d", cond_to_str(cond), immed); } std::string ARM_Disasm::DisassembleCLZ(uint32_t insn) |