<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 8, 2016 at 12:16 PM, 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"><span class="">On Mon, Feb 8, 2016 at 3:13 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
>> +   if (info->index_map[NIR_INTRINSIC_BASE] ||<br>
>> +       info->index_map[NIR_INTRINSIC_WRMASK]) {<br>
>> +      fprintf(fp, " /*");<br>
>> +      if (info->index_map[NIR_INTRINSIC_BASE])<br>
>> +         fprintf(fp, " base=%d", nir_intrinsic_base(instr));<br>
>> +      if (info->index_map[NIR_INTRINSIC_WRMASK]) {<br>
>> +          unsigned wrmask = nir_intrinsic_write_mask(instr);<br>
>> +          fprintf(fp, " wrmask=");<br>
>> +          for (unsigned i = 0; i < 4; i++)<br>
>> +             if ((wrmask >> i) & 1)<br>
>> +                fprintf(fp, "%c", "xyzw"[i]);<br>
>> +      }<br>
>> +      fprintf(fp, " */");<br>
>> +   }<br>
><br>
><br>
> Can we use an enum -> name table to name all of them?  Right now, it looks<br>
> like it only names baes and writemask.<br>
<br>
</span>Hmm.. sure.  I think writemask is really the only one needing special<br>
casing.  I guess all the others would just show as %d..<br></blockquote><div><br></div><div>Sure.  That one makes sense as hex.  The others shouldn't matter. <br></div></div><br></div></div>