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.

Attachment: pgpPgSrSOwDl2.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to