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

[Daniel Dunbar]

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
>