Thu Oct 27 06:15:47 PDT 2011
stored in the FogCoord vertex shader output. There is no fragment
shader change required. The fixed-function vertex program hasn't been
converted to the GLSL IR yet, only the fragment program, which is
Gallium only uses shaders, which means it always uses the generated
fixed-function programs (regardless of the IR being used), so
NV_fog_distance can always be enabled.
On Thu, Oct 27, 2011 at 3:18 PM, Jose Fonseca <jfonseca at vmware.com> wrote:
> I only see implementation for ffvertex_prog.c
> Wasn't (or will soon be) the FF->Mesa IR replaced with FF->GLSL IR? =C2=
=A0If so, can you truly enable unconditionally on gallium/st?
> ----- Original Message -----
>> If there are no other concerns, I'll push this on Saturday, including
>> the nv20 fix from Francisco Jerez.
>> On Tue, Sep 20, 2011 at 7:50 AM, Nicholas Miell <nmiell at gmail.com>
>> > On 09/19/2011 10:23 AM, Ian Romanick wrote:
>> >> I guess the big question is... why? =C2=A0With vertex shaders, this
>> >> functionality is not terribly useful. =C2=A0Over the past few weeks
>> >> we've
>> >> been *removing* code like this, so it seems kind of odd to add
>> >> some of
>> >> it. =C2=A0If I'm not mistaken, the only hardware in Mesa that can do
>> >> this
>> >> but not vertex shaders is NV10. =C2=A0Is it really that helpful?
>> >> I'm not necessarily opposed to adding this, I just want to be sure
>> >> we're giving it full thought... that's all.
>> > Good question. In the general case, I'd agree with you that
>> > implementing
>> > ancient extensions is a waste of effort, but this one is (AFAICT)
>> > an
>> > extremely low impact change that benefits a number of existing
>> > projects.
>> > It even has new code being written to target it, oddly enough
>> > (which is
>> > what prompted my interest in the first place).
>> > My first pass at this actually skipped the extension entirely and
>> > just
>> > changed the generated vertex program to always use the eye radial
>> > distance, but I figured that a full implementation of the extension
>> > would be the correct thing to do, especially when there may be some
>> > program out in the wild that breaks if the fog distance isn't
>> > abs(Ze).
>> > _______________________________________________
>> > mesa-dev mailing list
>> > mesa-dev at lists.freedesktop.org
>> > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
More information about the mesa-dev