Hi Dave,

i was not aware you are working on this actively as well, i had a series posted 5 days ago which got some critics and reviews [1]. The most important points where:

1. split functional change and renaming of the lowering pass [Ian]

2. check max clip/cull array sizes in link_shaders for all stages [Ian]

3. drop culldist semantics, which you already did [Ilia]


If you are interested in changes made to satisfy 1+2, you can fetch patches from here: https://git.thm.de/tjkl80/mesa.git arb-cull-distance


[1] https://lists.freedesktop.org/archives/mesa-dev/2016-May/115909.html

Greetings,

Tobias


On 13.05.2016 08:14, Dave Airlie wrote:
This is hopefully the final posting for this series, I've gotten
the lowering pass to look like I wanted, which is to say it lowers
to vec4[2].

TGSI then uses the CLIPDIST semantic and the two properties to
workout what is what. This means the CULLDIST semantic is no longer
required.

So I've ripped out CULLDIST from draw, and anywhere else it was used,
and fixed draw to use the new API, as it more closely reflects how
some of the hw works.

I've also fixed the array size maximum checks, however the piglit
test expects a link error when a compile error is a valid result.

Dave.

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

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

Reply via email to