On Thu, Aug 25, 2016 at 11:32 AM, Matt Turner <[email protected]> wrote:

> On Wed, Aug 24, 2016 at 10:49 PM, Jason Ekstrand <[email protected]>
> wrote:
> > In some programs, we can have very deep dominance trees and the recursion
> > can cause us to risk stack overflows.  Instead, we replace the recursion
> > with a pair of loops, one at the start and one at the end.  This is
> > functionally equivalent to what we had before and it's actually a bit
> > easier to read in the new form without the recursion.
> >
> > Signed-off-by: Jason Ekstrand <[email protected]>
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97225
>
> FWIW, I tested this without my patch, and it wasn't sufficient to
> avoid the problem.
>
> > Cc: [email protected]
>
> Not sure if necessary.
>

I'll drop it for now.  We can always backport later


> Regardless, with the other comments addressed
>
> Reviewed-by: Matt Turner <[email protected]>
>

Thanks!
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to