octo
/
citra
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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).
This commit is contained in:
Lioncash 2015-02-27 21:57:38 -05:00
parent c9ef377afa
commit 9ed3488925
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/core/arm/disassembler/arm_disasm.cpp
View File

@ -131,7 +131,7 @@ static const char *shift_names[] = {
"ROR" "ROR"
}; };
static const char* cond_to_str(int cond) { static const char* cond_to_str(uint32_t cond) {
return cond_names[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) std::string ARM_Disasm::DisassembleBKPT(uint32_t insn)
{ {
uint8_t cond = (insn >> 28) & 0xf;
uint32_t immed = (((insn >> 8) & 0xfff) << 4) | (insn & 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) std::string ARM_Disasm::DisassembleCLZ(uint32_t insn)