<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Oct 11, 2018 at 1:23 PM Ian Romanick <<a href="mailto:idr@freedesktop.org">idr@freedesktop.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 10/11/2018 11:21 AM, Ian Romanick wrote:<br>
> On 10/11/2018 08:38 AM, Jason Ekstrand wrote:<br>
>> ---<br>
>>  src/intel/compiler/brw_vec4_nir.cpp | 7 ++++++-<br>
>>  1 file changed, 6 insertions(+), 1 deletion(-)<br>
>><br>
>> diff --git a/src/intel/compiler/brw_vec4_nir.cpp b/src/intel/compiler/brw_vec4_nir.cpp<br>
>> index eaf1754b006..5ccfd1f8940 100644<br>
>> --- a/src/intel/compiler/brw_vec4_nir.cpp<br>
>> +++ b/src/intel/compiler/brw_vec4_nir.cpp<br>
>> @@ -1586,7 +1586,12 @@ vec4_visitor::nir_emit_alu(nir_alu_instr *instr)<br>
>>  <br>
>>     case nir_op_b2i:<br>
>>     case nir_op_b2f:<br>
>> -      emit(MOV(dst, negate(op[0])));<br>
>> +      if (nir_dest_bit_size(instr->dest.dest) > 32) {<br>
>> +         assert(dst.type == BRW_REGISTER_TYPE_DF);<br>
>> +         emit_conversion_to_double(dst, negate(op[0]), false);<br>
>> +      } else {<br>
>> +         emit(MOV(dst, negate(op[0])));<br>
>> +      }<br>
> <br>
> I wrote almost the same patch, but I split nir_op_b2i and nir_op_b2f<br>
> handling and omitted the assert.  I was never able to get the<br>
> then-clause to trigger.  Were you able to get that to happen?<br>
<br>
I guess the next patch causes it?  Assuming that's the case, you should<br>
add that to the commit message for this patch.<br></blockquote><div><br></div><div>Correct.  Sure, I can add that to the commit message.  I was going to CC it to stable because I don't really know if it's possible to trigger or not.  It definitely does trigger with the previous patch but I can't prove that it doesn't trigger without it.  Thoughts?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
>>        break;<br>
>>  <br>
>>     case nir_op_f2b:<br>
> <br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div></div>