On 06/30/2011 05:29 PM, Marek Olšák wrote:
> Hi,
> 
> this patch series implements ARB_depth_buffer_float in Mesa and Gallium. 
> There is complete r600g/r600-r700 support in my private branch, which passes 
> the same tests that pass for Z24S8. Softpipe has only sampler support. This 
> has turned out to be not so trivial, so it's possible I missed something.
> 
> I did not implement NV_depth_buffer_float, because it's not compatible with 
> the ARB variant. (GL_DEPTH_COMPONENT32F != GL_DEPTH_COMPONENT32F_NV etc.) The 
> NV extension can operate on unclamped depth values, whereas the ARB one 
> always clamps them.
> 
> Please review.

Marek,

You are awesome!  Thanks so much for implementing this!

Patches 1-7 look great:
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

I replied with a comment on patch 8, and am not planning on reviewing
patches 9-13 as I'm not familiar with Gallium.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to