diff options
| author | Marina Yatsina <marina.yatsina@intel.com> | 2016-01-21 11:37:06 +0000 |
|---|---|---|
| committer | Marina Yatsina <marina.yatsina@intel.com> | 2016-01-21 11:37:06 +0000 |
| commit | ff262fa8071f5680c794ec5bc2074ebec3eebce7 (patch) | |
| tree | 64745b848dce530dbba168196c54f32ab96b2fe3 /llvm/test | |
| parent | bfd29a1567ed3d208bcf45858199fb24a2eec52e (diff) | |
| download | bcm5719-llvm-ff262fa8071f5680c794ec5bc2074ebec3eebce7.tar.gz bcm5719-llvm-ff262fa8071f5680c794ec5bc2074ebec3eebce7.zip | |
[X86] - Removing warning on legal cases caused by commit r258132
There's an overloading of the "movsd" and "cmpsd" instructions, e.g. movsd can be either "Move Data from String to String" or "Move or Merge Scalar Double-Precision Floating-Point Value".
The former should produce warnings when parsing a memory operand that is not ESI/EDI, but the latter should not.
Fixed the code to produce warnings only after making sure we're dealing with the first case.
Expanded the tests of the produced warnings + fixed RUN line of the test so that it would check both stdout and stderr
Differential Revision: http://reviews.llvm.org/D16359
llvm-svn: 258393
Diffstat (limited to 'llvm/test')
| -rw-r--r-- | llvm/test/MC/X86/intel-syntax.s | 17 |
1 files changed, 16 insertions, 1 deletions
diff --git a/llvm/test/MC/X86/intel-syntax.s b/llvm/test/MC/X86/intel-syntax.s index 11dc8cf85bd..25bd1f284bb 100644 --- a/llvm/test/MC/X86/intel-syntax.s +++ b/llvm/test/MC/X86/intel-syntax.s @@ -1,4 +1,6 @@ -// RUN: llvm-mc -triple x86_64-unknown-unknown -x86-asm-syntax=intel %s | FileCheck %s +// RUN: llvm-mc -triple x86_64-unknown-unknown -x86-asm-syntax=intel %s > %t 2> %t.err +// RUN: FileCheck < %t %s +// RUN: FileCheck --check-prefix=CHECK-STDERR < %t.err %s _test: xor EAX, EAX @@ -755,26 +757,39 @@ sidt fword ptr [eax] ins byte ptr [eax], dx // CHECK: insb %dx, %es:(%edi) // CHECK-STDERR: memory operand is only for determining the size, ES:(R|E)DI will be used for the location +// CHECK-STDERR-NEXT: ins byte ptr [eax], dx outs dx, word ptr [eax] // CHECK: outsw (%esi), %dx // CHECK-STDERR: memory operand is only for determining the size, ES:(R|E)SI will be used for the location +// CHECK-STDERR-NEXT: outs dx, word ptr [eax] lods dword ptr [eax] // CHECK: lodsl (%esi), %eax // CHECK-STDERR: memory operand is only for determining the size, ES:(R|E)SI will be used for the location +// CHECK-STDERR-NEXT: lods dword ptr [eax] stos qword ptr [eax] // CHECK: stosq %rax, %es:(%edi) // CHECK-STDERR: memory operand is only for determining the size, ES:(R|E)DI will be used for the location +// CHECK-STDERR-NEXT: stos qword ptr [eax] scas byte ptr [eax] // CHECK: scasb %es:(%edi), %al // CHECK-STDERR: memory operand is only for determining the size, ES:(R|E)DI will be used for the location +// CHECK-STDERR-NEXT: scas byte ptr [eax] cmps word ptr [eax], word ptr [ebx] // CHECK: cmpsw %es:(%edi), (%esi) // CHECK-STDERR: memory operand is only for determining the size, ES:(R|E)SI will be used for the location +// CHECK-STDERR-NEXT: cmps word ptr [eax], word ptr [ebx] // CHECK-STDERR: memory operand is only for determining the size, ES:(R|E)DI will be used for the location +// CHECK-STDERR-NEXT: cmps word ptr [eax], word ptr [ebx] movs dword ptr [eax], dword ptr [ebx] // CHECK: movsl (%esi), %es:(%edi) // CHECK-STDERR: memory operand is only for determining the size, ES:(R|E)DI will be used for the location +// CHECK-STDERR-NEXT: movs dword ptr [eax], dword ptr [ebx] // CHECK-STDERR: memory operand is only for determining the size, ES:(R|E)SI will be used for the location +// CHECK-STDERR-NEXT: movs dword ptr [eax], dword ptr [ebx] + +movsd qword ptr [rax], xmm0 +// CHECK: movsd %xmm0, (%rax) +// CHECK-STDERR-NOT: movsd qword ptr [rax], xmm0 xlat byte ptr [eax] // CHECK: xlatb |
