i965: Don't emit MOVs with undefined registers for Gen4 point clipping.
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 28 Feb 2018 21:22:22 +0000 (13:22 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 28 Feb 2018 23:03:51 +0000 (15:03 -0800)
commite51b0664e03a028961e1a4250c49fbc3005b2fa4
treec9a4e642a9177c54306dd3339edeee2315809d25
parente4e79a02da2e813284aa8a82dfd4423f0ae9923a
i965: Don't emit MOVs with undefined registers for Gen4 point clipping.

Gen4 point clipping calls brw_clip_tri_alloc_regs with nr_verts == 0,
which means that c->reg.vertex[] isn't initialized.  It then emits MOVs
to stomp components of those uninitialized registers to 0.

This started causing assertions after Matt's recent series, when those
uninitialized registers started getting BRW_REGISTER_TYPE_NF, which
definitely doesn't exist on Gen4-5.

Reviewed-by: Matt Turner <mattst88@gmail.com>
src/intel/compiler/brw_clip_tri.c