<div dir="ltr"><div>I thought that was fairly obvious from "<var_name>", "(type *)thing", "thing->thing", and "thing[thing]"<br><br></div>I guess it's probably ok but it didn't seem necessary to me.  One thing I did sort-of want to do was to make it print the full deref back to the variable if possible.  I'll give a go at that and send out a fixup.<br><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Apr 7, 2018 at 9:13 AM, Rob Clark <span dir="ltr"><<a href="mailto:robdclark@gmail.com" target="_blank">robdclark@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Makes it more obvious what sort of deref instruction something is.<br>
<br>
Signed-off-by: Rob Clark <<a href="mailto:robdclark@gmail.com">robdclark@gmail.com</a>><br>
---<br>
 src/compiler/nir/nir_print.c | 8 +++++---<br>
 1 file changed, 5 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/src/compiler/nir/nir_print.c b/src/compiler/nir/nir_print.c<br>
index 6e6fd421c60..7d7542a4113 100644<br>
--- a/src/compiler/nir/nir_print.c<br>
+++ b/src/compiler/nir/nir_print.c<br>
@@ -494,17 +494,19 @@ print_deref_instr(nir_deref_<wbr>instr *instr, print_state *state)<br>
    print_dest(&instr->dest, state);<br>
<br>
    if (instr->deref_type == nir_deref_type_var) {<br>
-      fprintf(fp, " = deref %s", get_var_name(instr->var, state));<br>
+      fprintf(fp, " = deref_var %s", get_var_name(instr->var, state));<br>
       return;<br>
    } else if (instr->deref_type == nir_deref_type_cast) {<br>
-      fprintf(fp, " = deref (%s) (%s *)&",<br>
+      fprintf(fp, " = deref_cast (%s) (%s *)&",<br>
               get_variable_mode_str(instr-><wbr>mode),<br>
               glsl_get_type_name(instr-><wbr>type));<br>
       print_src(&instr->parent, state);<br>
       return;<br>
    }<br>
<br>
-   fprintf(fp, " = deref (%s) &", get_variable_mode_str(instr-><wbr>mode));<br>
+   fprintf(fp, " = deref_%s (%s) &",<br>
+          (instr->deref_type == nir_deref_type_struct) ? "struct" : "array",<br>
+          get_variable_mode_str(instr-><wbr>mode));<br>
    print_src(&instr->parent, state);<br>
<br>
    assert(instr->parent.is_ssa);<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.14.3<br>
<br>
</font></span></blockquote></div><br></div></div></div></div>