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

Tom Stellard tom at stellard.net
Tue Jan 22 11:24:47 PST 2013


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 Dänzer <michel.daenzer at amd.com> > > No piglit regressions with radeonsi. :)
> >> > 
> >> P.S. A couple of piglit tests still crash in control flow related code 
> >> though, e.g. glean/glsl1-do-loop or shaders/glsl-fs-discard-04. Have you 
> >> looked at these before, or would you mind taking a quick look? 
> > Oh, thx. I already wanted to ask what's next for radeonsi. I 
> >haven't realized that those still doesn't work, the last time I 
> >checked glsl1-do-loop was missing something else beside 
> >flow control. Going to dig into it, Christian. 
> Not sure if its related, but there's a clover for-loop test that triggers GPU resets on my HD6850...  Looks like an infinite loop.  Fixing the glsl1-do-loop might also fix the clover test.

This bug is unrelated, but it should be fixed once I get the new CFG
Structurizer working on R600.  These patches will help with that.

-Tom


More information about the mesa-dev mailing list