Re: [Mesa-dev] [PATCH 1/2] R600: handle loops to self in the structurizer v2

2013-01-23 Thread Tom Stellard
On Mon, Jan 21, 2013 at 10:28:56PM +0100, Christian König wrote: v2: don't mess up other loops Hi Christian, This patch regresses the glsl1-do-loop test, this test worked fine with v1 of the patch, but the structurizer creates an infinite loop with v2. See attached LLVM IR. -Tom

Re: [Mesa-dev] [PATCH 1/2] R600: handle loops to self in the structurizer v2

2013-01-22 Thread Michel Dänzer
On Mon, 2013-01-21 at 22:28 +0100, Christian König wrote: v2: don't mess up other loops Signed-off-by: Christian König deathsim...@vodafone.de Series is Tested-by: Michel Dänzer michel.daen...@amd.com No piglit regressions with radeonsi. :) P.S. A couple of piglit tests still crash in

Re: [Mesa-dev] [PATCH 1/2] R600: handle loops to self in the structurizer v2

2013-01-22 Thread Christian König
Am 22.01.2013 13:08, schrieb Michel Dänzer: On Mon, 2013-01-21 at 22:28 +0100, Christian König wrote: v2: don't mess up other loops Signed-off-by: Christian König deathsim...@vodafone.de Series is Tested-by: Michel Dänzer michel.daen...@amd.com No piglit regressions with radeonsi. :) P.S.

Re: [Mesa-dev] [PATCH 1/2] R600: handle loops to self in the structurizer v2

2013-01-22 Thread Aaron Watry
Am 22.01.2013 13:08, schrieb Michel Dänzer: On Mon, 2013-01-21 at 22:28 +0100, Christian König wrote: v2: don't mess up other loops Signed-off-by: Christian König deathsimple at vodafone.de Series is Tested-by: Michel Dänzer michel.daenzer at amd.com No piglit regressions with

Re: [Mesa-dev] [PATCH 1/2] R600: handle loops to self in the structurizer v2

2013-01-22 Thread Tom Stellard
On Tue, Jan 22, 2013 at 09:33:46AM -0600, Aaron Watry wrote: Am 22.01.2013 13:08, schrieb Michel Dänzer: On Mon, 2013-01-21 at 22:28 +0100, Christian König wrote: v2: don't mess up other loops Signed-off-by: Christian König deathsimple at vodafone.de Series is Tested-by: Michel

[Mesa-dev] [PATCH 1/2] R600: handle loops to self in the structurizer v2

2013-01-21 Thread Christian König
v2: don't mess up other loops Signed-off-by: Christian König deathsim...@vodafone.de --- lib/Target/R600/AMDGPUStructurizeCFG.cpp | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/lib/Target/R600/AMDGPUStructurizeCFG.cpp b/lib/Target/R600/AMDGPUStructurizeCFG.cpp