[cfe-commits] r129464 - in /cfe/trunk: lib/CodeGen/CGBuiltin.cpp test/CodeGen/builtins-x86.c

[Bill Wendling]

Oh...Okay. I'll remove them.

-bw

On Apr 20, 2011, at 3:34 PM, Daniel Dunbar wrote:

> Hi Bill,
> 
> Shouldn't these also be removed from BuiltinsX86.def? Currently, code
> that uses them will get a "cannot codegen this builtin yet" error.
> 
> - Daniel
> 
> On Wed, Apr 13, 2011 at 1:17 PM, Bill Wendling <isanbard at gmail.com> wrote:
>> Author: void
>> Date: Wed Apr 13 15:17:22 2011
>> New Revision: 129464
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=129464&view=rev
>> Log:
>> Removing the unaligned load tests from builtins-x86.c since they're generated by a regular 'load' now.
>> 
>> Modified:
>>    cfe/trunk/lib/CodeGen/CGBuiltin.cpp
>>    cfe/trunk/test/CodeGen/builtins-x86.c
>> 
>> Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGBuiltin.cpp?rev=129464&r1=129463&r2=129464&view=diff
>> ==============================================================================
>> --- cfe/trunk/lib/CodeGen/CGBuiltin.cpp (original)
>> +++ cfe/trunk/lib/CodeGen/CGBuiltin.cpp Wed Apr 13 15:17:22 2011
>> @@ -2092,8 +2092,6 @@
>>     // If palignr is shifting the pair of vectors more than 32 bytes, emit zero.
>>     return llvm::Constant::getNullValue(ConvertType(E->getType()));
>>   }
>> -  case X86::BI__builtin_ia32_loadups:
>> -  case X86::BI__builtin_ia32_loadupd:
>>   case X86::BI__builtin_ia32_loaddqu: {
>>     const llvm::Type *VecTy = ConvertType(E->getType());
>>     const llvm::Type *IntTy = llvm::IntegerType::get(getLLVMContext(), 128);
>> 
>> Modified: cfe/trunk/test/CodeGen/builtins-x86.c
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/builtins-x86.c?rev=129464&r1=129463&r2=129464&view=diff
>> ==============================================================================
>> --- cfe/trunk/test/CodeGen/builtins-x86.c (original)
>> +++ cfe/trunk/test/CodeGen/builtins-x86.c Wed Apr 13 15:17:22 2011
>> @@ -273,7 +273,6 @@
>>  #endif
>>   tmp_V2i = __builtin_ia32_cvttps2pi(tmp_V4f);
>>   (void) __builtin_ia32_maskmovq(tmp_V8c, tmp_V8c, tmp_cp);
>> -  tmp_V4f = __builtin_ia32_loadups(tmp_fCp);
>>   (void) __builtin_ia32_storeups(tmp_fp, tmp_V4f);
>>   (void) __builtin_ia32_storehps(tmp_V2ip, tmp_V4f);
>>   (void) __builtin_ia32_storelps(tmp_V2ip, tmp_V4f);
>> @@ -291,7 +290,6 @@
>>   tmp_V4f = __builtin_ia32_sqrtps(tmp_V4f);
>>   tmp_V4f = __builtin_ia32_sqrtss(tmp_V4f);
>>   (void) __builtin_ia32_maskmovdqu(tmp_V16c, tmp_V16c, tmp_cp);
>> -  tmp_V2d = __builtin_ia32_loadupd(tmp_dCp);
>>   (void) __builtin_ia32_storeupd(tmp_dp, tmp_V2d);
>>   tmp_i = __builtin_ia32_movmskpd(tmp_V2d);
>>   tmp_i = __builtin_ia32_pmovmskb128(tmp_V16c);
>> 
>> 
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>