[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