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.pohjolai...@intel.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
