[llvm-commits] [llvm] r60973 - in /llvm/trunk: lib/Transforms/Scalar/GVN.cpp test/Transforms/GVN/2007-07-26-InterlockingLoops.ll test/Transforms/GVN/2007-07-26-PhiErasure.ll test/Transforms/GVN/2007-07-31-NoDomInherit.ll test/Transforms/GVN/nonlocal.ll test/Transforms/GVN/rle-must-alias.ll test/Transforms/GVN/semidominated.ll

[Bill Wendling]

On Dec 12, 2008, at 11:22 PM, Chris Lattner wrote:

> Author: lattner
> Date: Sat Dec 13 01:22:47 2008
> New Revision: 60973
>
> URL: http://llvm.org/viewvc/llvm-project?rev=60973&view=rev
> Log:
> make RLE preserve the name of the load that it replaces.  This is just
> a pretification of the IR.
>
Chris,

This patch is inexplicably causing a failure for self-hosted LLVM in  
release mode. See here for the failure:

http://bwendling.apple.com:8020/builders/full-llvm-OSX/builds/180/steps/shell/logs/stdio

I temporarily reverted this.

-bw

> Modified:
>    llvm/trunk/lib/Transforms/Scalar/GVN.cpp
>    llvm/trunk/test/Transforms/GVN/2007-07-26-InterlockingLoops.ll
>    llvm/trunk/test/Transforms/GVN/2007-07-26-PhiErasure.ll
>    llvm/trunk/test/Transforms/GVN/2007-07-31-NoDomInherit.ll
>    llvm/trunk/test/Transforms/GVN/nonlocal.ll
>    llvm/trunk/test/Transforms/GVN/rle-must-alias.ll
>    llvm/trunk/test/Transforms/GVN/semidominated.ll
>
> Modified: llvm/trunk/lib/Transforms/Scalar/GVN.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/GVN.cpp?rev=60973&r1=60972&r2=60973&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/lib/Transforms/Scalar/GVN.cpp (original)
> +++ llvm/trunk/lib/Transforms/Scalar/GVN.cpp Sat Dec 13 01:22:47 2008
> @@ -1034,6 +1034,7 @@
>     // Perform PHI construction.
>     Value* v = GetValueForBlock(LI->getParent(), LI,  
> BlockReplValues, true);
>     LI->replaceAllUsesWith(v);
> +    v->takeName(LI);
>     if (isa<PointerType>(v->getType()))
>       MD->invalidateCachedPointerInfo(v);
>     toErase.push_back(LI);
>
> Modified: llvm/trunk/test/Transforms/GVN/2007-07-26- 
> InterlockingLoops.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GVN/2007-07-26-InterlockingLoops.ll?rev=60973&r1=60972&r2=60973&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/test/Transforms/GVN/2007-07-26-InterlockingLoops.ll  
> (original)
> +++ llvm/trunk/test/Transforms/GVN/2007-07-26-InterlockingLoops.ll  
> Sat Dec 13 01:22:47 2008
> @@ -1,5 +1,5 @@
> -; RUN: llvm-as < %s | opt -gvn | llvm-dis | not grep {tmp17625 =}
> -; RUN: llvm-as < %s | opt -gvn | llvm-dis | not grep {tmp17631 =}
> +; RUN: llvm-as < %s | opt -gvn | llvm-dis | grep {tmp17625.* = phi  
> i32. }
> +; RUN: llvm-as < %s | opt -gvn | llvm-dis | grep {tmp17631 = phi  
> i32. }
>
> @last = external global [65 x i32*]		; <[65 x i32*]*> [#uses=1]
>
>
> Modified: llvm/trunk/test/Transforms/GVN/2007-07-26-PhiErasure.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GVN/2007-07-26-PhiErasure.ll?rev=60973&r1=60972&r2=60973&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/test/Transforms/GVN/2007-07-26-PhiErasure.ll (original)
> +++ llvm/trunk/test/Transforms/GVN/2007-07-26-PhiErasure.ll Sat Dec  
> 13 01:22:47 2008
> @@ -1,4 +1,4 @@
> -; RUN: llvm-as < %s | opt -gvn | llvm-dis | not grep {tmp298316 =}
> +; RUN: llvm-as < %s | opt -gvn | llvm-dis | grep {tmp298316 = phi  
> i32 }
>
> 	%struct..0anon = type { i32 }
> 	%struct.FILE = type { i8*, i32, i32, i16, i16, %struct.__sbuf, i32,  
> i8*, i32 (i8*)*, i32 (i8*, i8*, i32)*, i64 (i8*, i64, i32)*, i32  
> (i8*, i8*, i32)*, %struct.__sbuf, %struct.__sFILEX*, i32, [3 x i8],  
> [1 x i8], %struct.__sbuf, i32, i64 }
>
> Modified: llvm/trunk/test/Transforms/GVN/2007-07-31-NoDomInherit.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GVN/2007-07-31-NoDomInherit.ll?rev=60973&r1=60972&r2=60973&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/test/Transforms/GVN/2007-07-31-NoDomInherit.ll  
> (original)
> +++ llvm/trunk/test/Transforms/GVN/2007-07-31-NoDomInherit.ll Sat  
> Dec 13 01:22:47 2008
> @@ -1,4 +1,4 @@
> -; RUN: llvm-as < %s | opt -gvn | llvm-dis | grep tmp51.rle
> +; RUN: llvm-as < %s | opt -gvn | llvm-dis | grep {tmp47 = phi i32 }
>
> 	%struct.anon = type { i32 (i32, i32, i32)*, i32, i32, [3 x i32],  
> i8*, i8*, i8* }
> @debug = external constant i32		; <i32*> [#uses=0]
>
> Modified: llvm/trunk/test/Transforms/GVN/nonlocal.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GVN/nonlocal.ll?rev=60973&r1=60972&r2=60973&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/test/Transforms/GVN/nonlocal.ll (original)
> +++ llvm/trunk/test/Transforms/GVN/nonlocal.ll Sat Dec 13 01:22:47  
> 2008
> @@ -1,4 +1,4 @@
> -; RUN: llvm-as < %s | opt -gvn | llvm-dis | not grep {DEAD =}
> +; RUN: llvm-as < %s | opt -gvn | llvm-dis | grep {%DEAD = phi i32. }
>
> define i32 @main(i32** %p) {
> block1:
>
> Modified: llvm/trunk/test/Transforms/GVN/rle-must-alias.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GVN/rle-must-alias.ll?rev=60973&r1=60972&r2=60973&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/test/Transforms/GVN/rle-must-alias.ll (original)
> +++ llvm/trunk/test/Transforms/GVN/rle-must-alias.ll Sat Dec 13  
> 01:22:47 2008
> @@ -1,4 +1,4 @@
> -; RUN: llvm-as < %s | opt -gvn | llvm-dis | grep {DEAD.rle = phi i32}
> +; RUN: llvm-as < %s | opt -gvn | llvm-dis | grep {DEAD = phi i32 }
> ; The %7 and %4 loads combine to make %DEAD unneeded.
> target datalayout = "e-p:32:32:32-i1:8:8-i8:8:8-i16:16:16-i32:32:32- 
> i64:32:64-f32:32:32-f64:32:64-v64:64:64-v128:128:128-a0:0:64- 
> f80:128:128"
> target triple = "i386-apple-darwin7"
>
> Modified: llvm/trunk/test/Transforms/GVN/semidominated.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GVN/semidominated.ll?rev=60973&r1=60972&r2=60973&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/test/Transforms/GVN/semidominated.ll (original)
> +++ llvm/trunk/test/Transforms/GVN/semidominated.ll Sat Dec 13  
> 01:22:47 2008
> @@ -1,4 +1,4 @@
> -; RUN: llvm-as < %s | opt -gvn | llvm-dis | not grep {DEAD =}
> +; RUN: llvm-as < %s | opt -gvn | llvm-dis | grep {DEAD = phi i32 }
>
> define i32 @main(i32* %p) {
> block1:
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits