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 <[email protected]> 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 >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
