[Mesa-dev] [PATCH 15/20] i965/cfg: Point to bblock_t containing associated control flow

Pohjolainen, Topi topi.pohjolainen at intel.com
Tue Aug 19 01:27:36 PDT 2014


On Thu, Jul 24, 2014 at 07:54:22PM -0700, Matt Turner wrote:
> ... rather than pointing directly to the associated instruction. This
> will let us set the block containing the IF statement's else-pointer to
> NULL, when we delete a useless ELSE instruction, as in the case
> 
>    (+f0) if(8)
>    ...
>    else(8)
>    endif(8)
> 
> Also, remove the pointer to the ENDIF, since it's unused, and it was
> also potentially wrong, in the case of a basic block containing both an
> ENDIF and an IF instruction:
> 
>    endif(8)
>    cmp.ne.f0(8) ...
>    (+f0) if(8)
> ---
>  src/mesa/drivers/dri/i965/brw_cfg.cpp             | 28 ++++++++---------------
>  src/mesa/drivers/dri/i965/brw_cfg.h               | 10 ++++----
>  src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp |  4 ++--
>  3 files changed, 15 insertions(+), 27 deletions(-)
> 

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


More information about the mesa-dev mailing list