[llvm-commits] [llvm] r40128 - /llvm/trunk/test/CFrontend/2004-02-13-Memset.c

[Chris Lattner]

> URL: http://llvm.org/viewvc/llvm-project?rev=40128&view=rev
> Log:
> Need -O1 or better to have these builtins lowered to
> llvm intrinsics in llvm-gcc-4.2.  This is because
> get_pointer_alignment bails out: it relies on TER
> to compute accurate alignment information.

This is fine.  However, I'm curious.  Do you mean it fails due to the  
calls to get_pointer_alignment in TreeToLLVM::EmitBuiltinMemCopy?  If  
so, we should fix TreeToLLVM::EmitBuiltinMemCopy to treat a null  
return as alignment = 1.

-Chris

> Modified:
>     llvm/trunk/test/CFrontend/2004-02-13-Memset.c
>
> Modified: llvm/trunk/test/CFrontend/2004-02-13-Memset.c
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CFrontend/ 
> 2004-02-13-Memset.c?rev=40128&r1=40127&r2=40128&view=diff
>
> ====================================================================== 
> ========
> --- llvm/trunk/test/CFrontend/2004-02-13-Memset.c (original)
> +++ llvm/trunk/test/CFrontend/2004-02-13-Memset.c Fri Jul 20  
> 17:39:06 2007
> @@ -1,4 +1,5 @@
> -// RUN: %llvmgcc -xc %s -c -o - | llvm-dis | grep llvm.memset | wc  
> -l | grep 3
> +// RUN: %llvmgcc -xc %s -c -O1 -o - | llvm-dis | grep llvm.memset | \
> +// RUN:   wc -l | grep 3
>
>  void test(int* X, char *Y) {
>    memset(X, 4, 1000);
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits