Keith Whitwell wrote:
Ian Romanick wrote:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
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 will probably a bit before I can add support to the parser. I'm
still trying to understand how src/mesa/shader/grammar works (and why
that was used instead of flex & byacc).
I've not yet committed this patch because it 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. :(
For the purposes of the integration work, I wouldn't be unhappy to see
the t_vb_arbprogram.* code turned off temporarily and let the
nvvertexec.c executor do the job in the interim.
I say this for a few reasons - one that I'm expecting great things from
your college-project optimizer, and second that if I were to continue
with t_vb_arbprogram.*, there would be a signficiant rework based on
what I've learned so far, and thirdly, it will make the ongoing reorg
easier in the meantime.
Oh, and while we're talking about archiving code, all the TCC stuff for fragment
programs can definitely be put in that category.
Keith
-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server. Download
it for free - -and be entered to win a 42" plasma tv or your very own
Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
