diff options
| author | Razvan A Lupusoru <razvan.a.lupusoru@intel.com> | 2014-01-29 16:02:57 -0800 |
|---|---|---|
| committer | Ian Rogers <irogers@google.com> | 2014-02-05 22:42:21 -0800 |
| commit | 2c498d1f28e62e81fbdb477ff93ca7454e7493d7 (patch) | |
| tree | 94654433a4dae83ab75d432304dcc0358aefeb1c /disassembler | |
| parent | 1dcff62155e8477eb114c8a86eb1beb0797ffc11 (diff) | |
| download | art-2c498d1f28e62e81fbdb477ff93ca7454e7493d7.zip art-2c498d1f28e62e81fbdb477ff93ca7454e7493d7.tar.gz art-2c498d1f28e62e81fbdb477ff93ca7454e7493d7.tar.bz2 | |
Specializing x86 range argument copying
The ARM implementation of range argument copying was specialized in some cases.
For all other architectures, it would fall back to generating memcpy. This patch
updates the x86 implementation so it does not call memcpy and instead generates
loads and stores, favoring movement of 128-bit chunks.
Change-Id: Ic891e5609a4b0e81a47c29cc5a9b301bd10a1933
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>
Diffstat (limited to 'disassembler')
| -rw-r--r-- | disassembler/disassembler_x86.cc | 36 |
1 files changed, 36 insertions, 0 deletions
diff --git a/disassembler/disassembler_x86.cc b/disassembler/disassembler_x86.cc index 6c25e0a..903d755 100644 --- a/disassembler/disassembler_x86.cc +++ b/disassembler/disassembler_x86.cc @@ -246,6 +246,42 @@ DISASSEMBLER_ENTRY(cmp, load = *instr == 0x10; store = !load; break; + case 0x12: case 0x13: + if (prefix[2] == 0x66) { + opcode << "movlpd"; + prefix[2] = 0; // clear prefix now it's served its purpose as part of the opcode + } else if (prefix[0] == 0) { + opcode << "movlps"; + } + has_modrm = true; + src_reg_file = dst_reg_file = SSE; + load = *instr == 0x12; + store = !load; + break; + case 0x16: case 0x17: + if (prefix[2] == 0x66) { + opcode << "movhpd"; + prefix[2] = 0; // clear prefix now it's served its purpose as part of the opcode + } else if (prefix[0] == 0) { + opcode << "movhps"; + } + has_modrm = true; + src_reg_file = dst_reg_file = SSE; + load = *instr == 0x16; + store = !load; + break; + case 0x28: case 0x29: + if (prefix[2] == 0x66) { + opcode << "movapd"; + prefix[2] = 0; // clear prefix now it's served its purpose as part of the opcode + } else if (prefix[0] == 0) { + opcode << "movaps"; + } + has_modrm = true; + src_reg_file = dst_reg_file = SSE; + load = *instr == 0x28; + store = !load; + break; case 0x2A: if (prefix[2] == 0x66) { opcode << "cvtpi2pd"; |