Timothy Arceri <tarc...@itsqueeze.com> writes:
> We won't split varyings marked as always active because there
> is no point in doing so. This means we need to mark both
> sides of the interface as always active otherwise we will have
> a mismatch and start removing things we shouldn't.
> ---
> src/compiler/glsl/link_varyings.cpp | 22 +++++++++++++++++++---
> 1 file changed, 19 insertions(+), 3 deletions(-)
>
> diff --git a/src/compiler/glsl/link_varyings.cpp
> b/src/compiler/glsl/link_varyings.cpp
> index ed3bf416878..e77ea79e2ba 100644
> --- a/src/compiler/glsl/link_varyings.cpp
> +++ b/src/compiler/glsl/link_varyings.cpp
> @@ -2268,9 +2268,6 @@ assign_varying_locations(struct gl_context *ctx,
> }
> }
>
> - _mesa_hash_table_destroy(consumer_inputs, NULL);
> - _mesa_hash_table_destroy(consumer_interface_inputs, NULL);
> -
> for (unsigned i = 0; i < num_tfeedback_decls; ++i) {
> if (!tfeedback_decls[i].is_varying())
> continue;
> @@ -2286,12 +2283,31 @@ assign_varying_locations(struct gl_context *ctx,
> /* Mark xfb varyings as always active */
> matched_candidate->toplevel_var->data.always_active_io = 1;
>
> + /* Mark the any corresponding inputs as always active also. We must do
> + * this because we have a NIR pass that lowers vertors to scalars and*vectors Other than that, Reviewed-by: Eric Anholt <e...@anholt.net>
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
