<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Oct 22, 2018 at 6:20 PM Alyssa Rosenzweig <<a href="mailto:alyssa@rosenzweig.io">alyssa@rosenzweig.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">For what it's worth, Midgard has real integers (including int32<br>
support), using hardware-level D3D10 boolean conventions. I'm trying to<br>
wrap my head around how this interacts with 5d85a0a.<br></blockquote><div><br></div><div>Right.  Sorry.  I forgot who's working on what chip these days.  Maybe Ilia can CC the right person.</div><div><br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm tempted to think the standard lower_bool_to_int32 pass would work,<br>
with an emulated b2f instruction in the backend IR level, rather than<br>
burdening NIR with those details.<br>
<br>
I'll have to look at the patch set closer to understand the impact. <br>
<br>
---<br>
<br>
> +nir_lower_bool_to_int32_impl(nir_function_impl *impl)<br>
> +nir_lower_bool_to_int32(nir_shader *shader)<br>
> +      if (function->impl && nir_lower_bool_to_int32_impl(function->impl))<br>
<br>
I'm guessing these were intended to read float32?<br>
</blockquote></div></div>