[Mesa-dev] [PATCH] nir: Make the printer include nir_variable::location too.

Jason Ekstrand jason at jlekstrand.net
Thu Mar 5 20:37:07 PST 2015


Fine by me
Reviewed-by: Jason Ekstrand <jason.ekstrand at intel.com>

On Thu, Mar 5, 2015 at 8:26 PM, Kenneth Graunke <kenneth at whitecape.org>
wrote:

> Being able to see both location and driver_location can be useful when
> debugging IO mistakes.
>
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> ---
>  src/glsl/nir/nir_print.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/glsl/nir/nir_print.c b/src/glsl/nir/nir_print.c
> index 6a3c6a0..30d4821 100644
> --- a/src/glsl/nir/nir_print.c
> +++ b/src/glsl/nir/nir_print.c
> @@ -228,7 +228,7 @@ print_var_decl(nir_variable *var, print_var_state
> *state, FILE *fp)
>     if (var->data.mode == nir_var_shader_in ||
>         var->data.mode == nir_var_shader_out ||
>         var->data.mode == nir_var_uniform) {
> -      fprintf(fp, " (%u)", var->data.driver_location);
> +      fprintf(fp, " (%u, %u)", var->data.location,
> var->data.driver_location);
>     }
>
>     fprintf(fp, "\n");
> --
> 2.2.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150305/b917c9e2/attachment.html>


More information about the mesa-dev mailing list