[Mesa-dev] [PATCH 05/14] mesa/main/ff_frag: Don't bother with VARYING_BIT_FOGC.

[Gustaw Smolarczyk]

2017-05-15 20:28 GMT+02:00 Ian Romanick <idr at freedesktop.org>:
> With this patch I applied, I still see this bit used in several
> places, including the i915 driver.  Did you test that?
>
> rc/compiler/shader_enums.h:#define VARYING_BIT_FOGC BITFIELD64_BIT(VARYING_SLOT_FOGC)
> src/mesa/swrast/s_context.c:                    VARYING_BIT_FOGC | VARYING_BITS_TEX_ANY;
> src/mesa/swrast/s_context.c:         attribsMask |= VARYING_BIT_FOGC;
> src/mesa/swrast/s_fog.c:if (span->arrayAttribs & VARYING_BIT_FOGC) {                                    \
> src/mesa/program/programopt.c: * This function sets \c VARYING_BIT_FOGC in \c fprog->info.inputs_read.
> src/mesa/program/programopt.c:   fprog->info.inputs_read |= VARYING_BIT_FOGC;
> src/mesa/main/ffvertex_prog.c:   if (p->state->fragprog_inputs_read & VARYING_BIT_FOGC)
> src/mesa/drivers/dri/i915/i915_fragprog.c:   if ((inputsRead & VARYING_BIT_FOGC)) {
> src/mesa/tnl/t_context.c:       || (fp != NULL && (fp->info.inputs_read & VARYING_BIT_FOGC) != 0)) {

The bit would eventually end in the state_key::inputs_available bitset
which is only used locally (i.e. in the same source file). It was
never tested against.

Regards,
Gustaw