[Mesa-dev] [PATCH 03/28] glsl: don't change the varying type in validation code

Anuj Phogat anuj.phogat at gmail.com
Tue Jan 5 15:20:35 PST 2016


On Mon, Dec 28, 2015 at 9:00 PM, Timothy Arceri
<timothy.arceri at collabora.com> wrote:
> There is a function dedicated to demoting unused varyings lets
> trust it to do its job.
> ---
>  src/glsl/link_varyings.cpp | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
> index 9fe05ca..961c3d7 100644
> --- a/src/glsl/link_varyings.cpp
> +++ b/src/glsl/link_varyings.cpp
> @@ -1680,11 +1680,6 @@ assign_varying_locations(struct gl_context *ctx,
>                             var->name,
>                              _mesa_shader_stage_to_string(producer->Stage));
>              }
> -
> -            /* An 'in' variable is only really a shader input if its
> -             * value is written by the previous stage.
> -             */
> -            var->data.mode = ir_var_auto;
>           }
>        }
>     }
> --
> 2.4.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>


More information about the mesa-dev mailing list