[llvm] r186924 - Don't let x86 asm printer use the no operand movsd alias. It should use the normal movsl instead.
Craig Topper
craig.topper at gmail.com
Mon Jul 22 18:50:47 PDT 2013
Author: ctopper
Date: Mon Jul 22 20:50:47 2013
New Revision: 186924
URL: http://llvm.org/viewvc/llvm-project?rev=186924&view=rev
Log:
Don't let x86 asm printer use the no operand movsd alias. It should use the normal movsl instead.
Modified:
llvm/trunk/lib/Target/X86/X86InstrInfo.td
llvm/trunk/test/MC/Disassembler/X86/simple-tests.txt
llvm/trunk/test/MC/X86/x86-32.s
llvm/trunk/test/MC/X86/x86-64.s
Modified: llvm/trunk/lib/Target/X86/X86InstrInfo.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86InstrInfo.td?rev=186924&r1=186923&r2=186924&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86InstrInfo.td (original)
+++ llvm/trunk/lib/Target/X86/X86InstrInfo.td Mon Jul 22 20:50:47 2013
@@ -2128,7 +2128,7 @@ def : InstAlias<"movq $src, $dst",
// movsd with no operands (as opposed to the SSE scalar move of a double) is an
// alias for movsl. (as in rep; movsd)
-def : InstAlias<"movsd", (MOVSD)>;
+def : InstAlias<"movsd", (MOVSD), 0>;
// movsx aliases
def : InstAlias<"movsx $src, $dst", (MOVSX16rr8 GR16:$dst, GR8:$src), 0>;
Modified: llvm/trunk/test/MC/Disassembler/X86/simple-tests.txt
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/Disassembler/X86/simple-tests.txt?rev=186924&r1=186923&r2=186924&view=diff
==============================================================================
--- llvm/trunk/test/MC/Disassembler/X86/simple-tests.txt (original)
+++ llvm/trunk/test/MC/Disassembler/X86/simple-tests.txt Mon Jul 22 20:50:47 2013
@@ -756,7 +756,7 @@
# rdar://13493622 lldb doesn't print the x86 rep/repne prefix when disassembling
# CHECK: repne
-# CHECK-NEXT: movsd
+# CHECK-NEXT: movsl
0xf2 0xa5
# CHECK: repne
# CHECK-NEXT: movsq
Modified: llvm/trunk/test/MC/X86/x86-32.s
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/X86/x86-32.s?rev=186924&r1=186923&r2=186924&view=diff
==============================================================================
--- llvm/trunk/test/MC/X86/x86-32.s (original)
+++ llvm/trunk/test/MC/X86/x86-32.s Mon Jul 22 20:50:47 2013
@@ -894,9 +894,9 @@ pshufw $90, %mm4, %mm0
movsw %ds:(%esi), %es:(%edi)
movsw (%esi), %es:(%edi)
-// CHECK: movsd # encoding: [0xa5]
-// CHECK: movsd
-// CHECK: movsd
+// CHECK: movsl # encoding: [0xa5]
+// CHECK: movsl
+// CHECK: movsl
movsl
movsl %ds:(%esi), %es:(%edi)
movsl (%esi), %es:(%edi)
Modified: llvm/trunk/test/MC/X86/x86-64.s
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/X86/x86-64.s?rev=186924&r1=186923&r2=186924&view=diff
==============================================================================
--- llvm/trunk/test/MC/X86/x86-64.s (original)
+++ llvm/trunk/test/MC/X86/x86-64.s Mon Jul 22 20:50:47 2013
@@ -115,12 +115,12 @@
// rdar://8470918
smovb // CHECK: movsb
smovw // CHECK: movsw
-smovl // CHECK: movsd
+smovl // CHECK: movsl
smovq // CHECK: movsq
// rdar://8456361
// CHECK: rep
-// CHECK: movsd
+// CHECK: movsl
rep movsd
// CHECK: rep
@@ -627,7 +627,7 @@ movsq
// CHECK: encoding: [0x48,0xa5]
movsl
-// CHECK: movsd
+// CHECK: movsl
// CHECK: encoding: [0xa5]
stosq
@@ -845,7 +845,7 @@ lock/incl 1(%rsp)
rep movsl
// CHECK: rep
// CHECK: encoding: [0xf3]
-// CHECK: movsd
+// CHECK: movsl
// CHECK: encoding: [0xa5]
@@ -1083,14 +1083,14 @@ xsetbv // CHECK: xsetbv # encoding: [0x0
movsw %ds:(%rsi), %es:(%rdi)
movsw (%rsi), %es:(%rdi)
-// CHECK: movsd # encoding: [0xa5]
-// CHECK: movsd
-// CHECK: movsd
+// CHECK: movsl # encoding: [0xa5]
+// CHECK: movsl
+// CHECK: movsl
movsl
movsl %ds:(%rsi), %es:(%rdi)
movsl (%rsi), %es:(%rdi)
// rdar://10883092
-// CHECK: movsd
+// CHECK: movsl
movsl (%rsi), (%rdi)
// CHECK: movsq # encoding: [0x48,0xa5]
More information about the llvm-commits
mailing list