On Tue, 29 Nov 2011 16:08:39 +0800, Yuanhan Liu <yuanhan....@linux.intel.com> wrote: > Increase the brw eu instruction store size dynamically instead of just > allocating it statically with a constant limit. This would fix something > that 'GL_MAX_PROGRAM_INSTRUCTIONS_ARB was 16384 while the driver would > limit it to 10000'.
I was going to caution against the other assumptions on insn pointers staying valid in pre-gen6 code in case someone reduced the initial store size at some point, and you'd already reduced the initial store size :) So, in brw_clip_emit.c and brw_sf_emit.c there are a few cases (particularly loops) where the brw_instruction * is taken from one instruction emit and used later. I'd be comfortable with this patch if those were converted to using an instruction index as well.
pgpPgSrSOwDl2.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev