diff options
author | Ian Rogers <irogers@google.com> | 2014-04-25 21:36:59 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2014-04-25 21:37:00 +0000 |
commit | 7f40b111755e300ddddd6839425337fe3af8d4e7 (patch) | |
tree | 5a0cebe690d040795083b9051d378488bad7cc6f | |
parent | bb43b2102cf1f3a753448e684a5540654382e69c (diff) | |
parent | e8861b30ac8b2b1ca49386f9c9218f1d6fedc511 (diff) | |
download | art-7f40b111755e300ddddd6839425337fe3af8d4e7.zip art-7f40b111755e300ddddd6839425337fe3af8d4e7.tar.gz art-7f40b111755e300ddddd6839425337fe3af8d4e7.tar.bz2 |
Merge "ART: Enables x86_64 disassembly"
-rw-r--r-- | disassembler/disassembler_x86.cc | 32 |
1 files changed, 25 insertions, 7 deletions
diff --git a/disassembler/disassembler_x86.cc b/disassembler/disassembler_x86.cc index 1b96a2b..5cc6acf 100644 --- a/disassembler/disassembler_x86.cc +++ b/disassembler/disassembler_x86.cc @@ -145,6 +145,9 @@ size_t DisassemblerX86::DumpInstruction(std::ostream& os, const uint8_t* instr) } } while (have_prefixes); uint8_t rex = (supports_rex_ && (*instr >= 0x40) && (*instr <= 0x4F)) ? *instr : 0; + if (rex != 0) { + instr++; + } bool has_modrm = false; bool reg_is_opcode = false; size_t immediate_bytes = 0; @@ -746,14 +749,18 @@ DISASSEMBLER_ENTRY(cmp, uint8_t reg_or_opcode = (modrm >> 3) & 7; uint8_t rm = modrm & 7; std::ostringstream address; - if (mod == 0 && rm == 5) { // fixed address - address_bits = *reinterpret_cast<const uint32_t*>(instr); - address << StringPrintf("[0x%x]", address_bits); + if (mod == 0 && rm == 5) { + if (!supports_rex_) { // Absolute address. + address_bits = *reinterpret_cast<const uint32_t*>(instr); + address << StringPrintf("[0x%x]", address_bits); + } else { // 64-bit RIP relative addressing. + address << StringPrintf("[RIP + 0x%x]", *reinterpret_cast<const uint32_t*>(instr)); + } instr += 4; } else if (rm == 4 && mod != 3) { // SIB uint8_t sib = *instr; instr++; - uint8_t ss = (sib >> 6) & 3; + uint8_t scale = (sib >> 6) & 3; uint8_t index = (sib >> 3) & 7; uint8_t base = sib & 7; address << "["; @@ -765,11 +772,22 @@ DISASSEMBLER_ENTRY(cmp, } if (index != 4) { DumpIndexReg(address, rex, index); - if (ss != 0) { - address << StringPrintf(" * %d", 1 << ss); + if (scale != 0) { + address << StringPrintf(" * %d", 1 << scale); } } - if (mod == 1) { + if (mod == 0) { + if (base == 5) { + if (index != 4) { + address << StringPrintf(" + %d", *reinterpret_cast<const int32_t*>(instr)); + } else { + // 64-bit low 32-bit absolute address, redundant absolute address encoding on 32-bit. + address_bits = *reinterpret_cast<const uint32_t*>(instr); + address << StringPrintf("%d", address_bits); + } + instr += 4; + } + } else if (mod == 1) { address << StringPrintf(" + %d", *reinterpret_cast<const int8_t*>(instr)); instr++; } else if (mod == 2) { |