[Mesa-dev] [PATCH 20/20] i965: Add and use functions to get next/prev blocks.

Pohjolainen, Topi topi.pohjolainen at intel.com
Wed Sep 17 23:56:56 PDT 2014


On Thu, Sep 04, 2014 at 01:26:45PM -0700, Matt Turner wrote:
> On Tue, Sep 2, 2014 at 9:34 PM, Matt Turner <mattst88 at gmail.com> wrote:
> > diff --git a/src/mesa/drivers/dri/i965/brw_dead_control_flow.cpp b/src/mesa/drivers/dri/i965/brw_dead_control_flow.cpp
> > index 557c3ad..8a7f42a 100644
> > --- a/src/mesa/drivers/dri/i965/brw_dead_control_flow.cpp
> > +++ b/src/mesa/drivers/dri/i965/brw_dead_control_flow.cpp
> > @@ -52,20 +52,20 @@ dead_control_flow_eliminate(backend_visitor *v)
> >           continue;
> >
> >        backend_instruction *if_inst = NULL, *else_inst = NULL;
> > -      backend_instruction *prev_inst = ((bblock_t *)endif_block->link.prev)->end();
> > +      backend_instruction *prev_inst = endif_block->next()->end();
> 
> This is obviously supposed to be ->prev(), not ->next(). Fixed locally.

With that patches 19 and 20 are:

Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>


More information about the mesa-dev mailing list