[Mesa-dev] [PATCH] glsl: Fix indent in dump code

Ian Romanick idr at freedesktop.org
Wed Jun 21 00:26:47 UTC 2017


Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

On 03/06/2017 07:27 AM, Elie Tournier wrote:
> From: Elie Tournier <tournier.elie at gmail.com>
> 
> Signed-off-by: Elie Tournier <elie.tournier at collabora.com>
> ---
> On an other thread [1], we discovered than the GLSL compiler have some trouble to indent correctly the dump IR.
> 
> I don't have git access, please push it for me.
> 
> [1] https://lists.freedesktop.org/archives/mesa-dev/2017-March/146518.html
> ---
>  src/compiler/glsl/ir_builder_print_visitor.cpp | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/src/compiler/glsl/ir_builder_print_visitor.cpp b/src/compiler/glsl/ir_builder_print_visitor.cpp
> index 825dbe148e..02f15e74ee 100644
> --- a/src/compiler/glsl/ir_builder_print_visitor.cpp
> +++ b/src/compiler/glsl/ir_builder_print_visitor.cpp
> @@ -372,17 +372,17 @@ ir_builder_print_visitor::visit(ir_constant *ir)
>           switch (ir->type->base_type) {
>           case GLSL_TYPE_UINT:
>              if (ir->value.u[i] != 0)
> -               print_without_indent("r%04X_data.u[%u] = %u;\n",
> +               print_with_indent("r%04X_data.u[%u] = %u;\n",
>                                      my_index, i, ir->value.u[i]);
>              break;
>           case GLSL_TYPE_INT:
>              if (ir->value.i[i] != 0)
> -               print_without_indent("r%04X_data.i[%u] = %i;\n",
> +               print_with_indent("r%04X_data.i[%u] = %i;\n",
>                                      my_index, i, ir->value.i[i]);
>              break;
>           case GLSL_TYPE_FLOAT:
>              if (ir->value.u[i] != 0)
> -               print_without_indent("r%04X_data.u[%u] = 0x%08x; /* %f */\n",
> +               print_with_indent("r%04X_data.u[%u] = 0x%08x; /* %f */\n",
>                                      my_index,
>                                      i,
>                                      ir->value.u[i],
> @@ -395,27 +395,27 @@ ir_builder_print_visitor::visit(ir_constant *ir)
>  
>              memcpy(&v, &ir->value.d[i], sizeof(v));
>              if (v != 0)
> -               print_without_indent("r%04X_data.u64[%u] = 0x%016" PRIx64 "; /* %g */\n",
> +               print_with_indent("r%04X_data.u64[%u] = 0x%016" PRIx64 "; /* %g */\n",
>                                      my_index, i, v, ir->value.d[i]);
>              break;
>           }
>           case GLSL_TYPE_UINT64:
>              if (ir->value.u64[i] != 0)
> -               print_without_indent("r%04X_data.u64[%u] = %" PRIu64 ";\n",
> +               print_with_indent("r%04X_data.u64[%u] = %" PRIu64 ";\n",
>                                      my_index,
>                                      i,
>                                      ir->value.u64[i]);
>              break;
>           case GLSL_TYPE_INT64:
>              if (ir->value.i64[i] != 0)
> -               print_without_indent("r%04X_data.i64[%u] = %" PRId64 ";\n",
> +               print_with_indent("r%04X_data.i64[%u] = %" PRId64 ";\n",
>                                      my_index,
>                                      i,
>                                      ir->value.i64[i]);
>              break;
>           case GLSL_TYPE_BOOL:
>              if (ir->value.u[i] != 0)
> -               print_without_indent("r%04X_data.u[%u] = 1;\n", my_index, i);
> +               print_with_indent("r%04X_data.u[%u] = 1;\n", my_index, i);
>              break;
>           default:
>              unreachable("Invalid constant type");
> 



More information about the mesa-dev mailing list