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
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
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.
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
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
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