<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 3, 2017 at 3:10 PM, Connor Abbott <span dir="ltr"><<a href="mailto:cwabbott0@gmail.com" target="_blank">cwabbott0@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This, and patch 18 updated to delete the case in vtn_ssa_value() are:<br>
<br>
Reviewed-by: Connor Abbott <<a href="mailto:cwabbott0@gmail.com">cwabbott0@gmail.com</a>><br><div class="HOEnZb"><div class="h5"></div></div></blockquote><div><br></div><div>Thanks!  Pushed.<br><br></div><div>The one tiny change I made was to add a vtn_pointer_for_variable helper to ensure that things are always initialized correctly.<br><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"><div class="HOEnZb"><div class="h5">
..On Fri, Jun 30, 2017 at 10:30 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> Cc: Connor Abbott <<a href="mailto:cwabbott0@gmail.com">cwabbott0@gmail.com</a>><br>
><br>
> ---<br>
>  src/compiler/spirv/vtn_<wbr>glsl450.c | 7 ++++++-<br>
>  1 file changed, 6 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/compiler/spirv/vtn_<wbr>glsl450.c b/src/compiler/spirv/vtn_<wbr>glsl450.c<br>
> index 5d38431..cf49465 100644<br>
> --- a/src/compiler/spirv/vtn_<wbr>glsl450.c<br>
> +++ b/src/compiler/spirv/vtn_<wbr>glsl450.c<br>
> @@ -479,8 +479,13 @@ handle_glsl450_alu(struct vtn_builder *b, enum GLSLstd450 entrypoint,<br>
>     /* Collect the various SSA sources */<br>
>     unsigned num_inputs = count - 5;<br>
>     nir_ssa_def *src[3] = { NULL, };<br>
> -   for (unsigned i = 0; i < num_inputs; i++)<br>
> +   for (unsigned i = 0; i < num_inputs; i++) {<br>
> +      /* These are handled specially below */<br>
> +      if (vtn_untyped_value(b, w[i + 5])->value_type == vtn_value_type_pointer)<br>
> +         continue;<br>
> +<br>
>        src[i] = vtn_ssa_value(b, w[i + 5])->def;<br>
> +   }<br>
><br>
>     switch (entrypoint) {<br>
>     case GLSLstd450Radians:<br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</div></div></blockquote></div><br></div></div>