[Mesa3d-dev] Re: [Mesa3d-cvs] CVS Update: Mesa (branch: trunk)

[Keith W. <ke...@tu...>]

Ian Romanick wrote:
> CVSROOT:	/cvs/mesa
> Module name:	Mesa
> Repository:	Mesa/src/mesa/tnl/
> Changes by:	idr@gabe.	05/11/08 14:40:26
> 
> Log message:
>   This patch adds the missing NV_vertex_program2 and NV_vertex_program3
>   instructions to the various internal tables.  It does not add support for
>   them to the parser nor does it add support for them to the program
>   interpreter.
>   
>   It also corrects some errors in the comments in program_instruction.h.
>   
>   This patch breaks the instruction union in t_vb_arbprogram.h.  The opcode
>   field in that union was only 6 bits.  When there were only 59+3 instructions
>   that worked fine. However, this patch increases the base instruction count
>   to 69.  This requires 7 bits.  The problem is that increasing the opcode
>   width to 7 bits increases the size of instruction::rsw to 33 bits.  I've
>   increased the size of the union to 64-bit, but this is just a temporary hack
>   (and probably breaks the build on some systems).

Probably better to drop the 'dword' member altogther and just clear the 
instructions with memset when required...

Keith