[llvm-commits] [llvm] r136292 - in /llvm/trunk: lib/Target/ARM/AsmParser/ARMAsmParser.cpp lib/Target/ARM/MCTargetDesc/ARMMCCodeEmitter.cpp lib/Target/X86/AsmParser/X86AsmParser.cpp lib/Target/X86/MCTargetDesc/X86MCCodeEmitter.cpp test/MC/X86/3DNo

[Evan Cheng]

On Jul 27, 2011, at 5:22 PM, Eli Friedman <eli.friedman at gmail.com> wrote:

> On Wed, Jul 27, 2011 at 4:22 PM, Evan Cheng <evan.cheng at apple.com> wrote:
>> Modified: llvm/trunk/test/MC/X86/3DNow.s
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/X86/3DNow.s?rev=136292&r1=136291&r2=136292&view=diff
>> ==============================================================================
>> --- llvm/trunk/test/MC/X86/3DNow.s (original)
>> +++ llvm/trunk/test/MC/X86/3DNow.s Wed Jul 27 18:22:03 2011
>> @@ -1,18 +1,18 @@
>> -// RUN: llvm-mc -triple i386-unknown-unknown --show-encoding %s | FileCheck %s
>> +// RUN: llvm-mc -triple x86_64-unknown-unknown --show-encoding %s | FileCheck %s
>> 
>>  // PR8283
>> 
>>  // CHECK: pavgusb %mm2, %mm1  # encoding: [0x0f,0x0f,0xca,0xbf]
>>  pavgusb        %mm2, %mm1
>> 
>> -// CHECK: pavgusb 9(%esi,%edx), %mm3 # encoding: [0x0f,0x0f,0x5c,0x16,0x09,0xbf]
>> +// CHECK: pavgusb 9(%esi,%edx), %mm3 # encoding: [0x67,0x0f,0x0f,0x5c,0x16,0x09,0xbf]
>>  pavgusb        9(%esi,%edx), %mm3
>> 
>> 
>>  // CHECK: pf2id %mm2, %mm1  # encoding: [0x0f,0x0f,0xca,0x1d]
>>  pf2id  %mm2, %mm1
>> 
>> -// CHECK: pf2id 9(%esi,%edx), %mm3 # encoding: [0x0f,0x0f,0x5c,0x16,0x09,0x1d]
>> +// CHECK: pf2id 9(%esi,%edx), %mm3 # encoding: [0x67,0x0f,0x0f,0x5c,0x16,0x09,0x1d]
>>  pf2id  9(%esi,%edx), %mm3
> 
> Err, this seems like an odd place to be testing unusual address-size prefixes...

It's not. The tests were using 64-bit registers in 32-bit mode so I changed the triple. 

Evan

> 
> -Eli