summaryrefslogtreecommitdiff
path: root/compiler
diff options
context:
space:
mode:
author Nicolas Geoffray <ngeoffray@google.com> 2015-09-18 13:51:30 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2015-09-18 13:51:30 +0000
commit7bfd7ee880785ef383f6434eb4eb35fcaac5ad5a (patch)
tree42b5d6ea600edac939677ee8e021d91ac12da1c0 /compiler
parent819a9c5638b6d6b579c89fe36df96acc1f378182 (diff)
parent46fe0650be6a69f63b54c0967194350c6a145557 (diff)
Merge "Fix x64's cmpw."
Diffstat (limited to 'compiler')
-rw-r--r--compiler/utils/x86_64/assembler_x86_64.cc2
-rw-r--r--compiler/utils/x86_64/assembler_x86_64_test.cc26
2 files changed, 26 insertions, 2 deletions
diff --git a/compiler/utils/x86_64/assembler_x86_64.cc b/compiler/utils/x86_64/assembler_x86_64.cc
index 89d7915f04..6e7d74d528 100644
--- a/compiler/utils/x86_64/assembler_x86_64.cc
+++ b/compiler/utils/x86_64/assembler_x86_64.cc
@@ -1213,8 +1213,8 @@ void X86_64Assembler::xchgl(CpuRegister reg, const Address& address) {
void X86_64Assembler::cmpw(const Address& address, const Immediate& imm) {
AssemblerBuffer::EnsureCapacity ensured(&buffer_);
+ EmitOperandSizeOverride();
EmitOptionalRex32(address);
- EmitUint8(0x66);
EmitComplex(7, address, imm);
}
diff --git a/compiler/utils/x86_64/assembler_x86_64_test.cc b/compiler/utils/x86_64/assembler_x86_64_test.cc
index 82378f724a..00bb5ca36b 100644
--- a/compiler/utils/x86_64/assembler_x86_64_test.cc
+++ b/compiler/utils/x86_64/assembler_x86_64_test.cc
@@ -755,10 +755,34 @@ TEST_F(AssemblerX86_64Test, Movl) {
TEST_F(AssemblerX86_64Test, Movw) {
GetAssembler()->movw(x86_64::Address(x86_64::CpuRegister(x86_64::RAX), 0),
x86_64::CpuRegister(x86_64::R9));
- const char* expected = "movw %R9w, 0(%RAX)\n";
+ GetAssembler()->movw(x86_64::Address(x86_64::CpuRegister(x86_64::RAX), 0),
+ x86_64::Immediate(0));
+ GetAssembler()->movw(x86_64::Address(x86_64::CpuRegister(x86_64::R9), 0),
+ x86_64::Immediate(0));
+ GetAssembler()->movw(x86_64::Address(x86_64::CpuRegister(x86_64::R14), 0),
+ x86_64::Immediate(0));
+ const char* expected =
+ "movw %R9w, 0(%RAX)\n"
+ "movw $0, 0(%RAX)\n"
+ "movw $0, 0(%R9)\n"
+ "movw $0, 0(%R14)\n";
DriverStr(expected, "movw");
}
+TEST_F(AssemblerX86_64Test, Cmpw) {
+ GetAssembler()->cmpw(x86_64::Address(x86_64::CpuRegister(x86_64::RAX), 0),
+ x86_64::Immediate(0));
+ GetAssembler()->cmpw(x86_64::Address(x86_64::CpuRegister(x86_64::R9), 0),
+ x86_64::Immediate(0));
+ GetAssembler()->cmpw(x86_64::Address(x86_64::CpuRegister(x86_64::R14), 0),
+ x86_64::Immediate(0));
+ const char* expected =
+ "cmpw $0, 0(%RAX)\n"
+ "cmpw $0, 0(%R9)\n"
+ "cmpw $0, 0(%R14)\n";
+ DriverStr(expected, "cmpw");
+}
+
TEST_F(AssemblerX86_64Test, MovqAddrImm) {
GetAssembler()->movq(x86_64::Address(x86_64::CpuRegister(x86_64::RAX), 0),
x86_64::Immediate(-5));