<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 14, 2016 at 6:59 AM, Iago Toral <span dir="ltr"><<a href="mailto:itoral@igalia.com" target="_blank">itoral@igalia.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Both patches are:<br>
Reviewed-by: Iago Toral Quiroga <<a href="mailto:itoral@igalia.com">itoral@igalia.com</a>><br></blockquote><div><br></div><div>Thanks!<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"><br>
On Wed, 2016-07-13 at 14:28 -0700, Jason Ekstrand wrote:<br>
> It was returning true if the function types have different lengths<br>
> rather<br>
> than false.  This was new with the SPIR-V to NIR pass and I thought<br>
> I'd<br>
> fixed it a while ago but it may have gotten lost in rebasing<br>
> somewhere.<br>
><br>
> Signed-off-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
> Cc: "12.0" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.org</a>><br>
> ---<br>
>  src/compiler/glsl_types.cpp | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/src/compiler/glsl_types.cpp<br>
> b/src/compiler/glsl_types.cpp<br>
> index 066a74e..fa27135 100644<br>
> --- a/src/compiler/glsl_types.cpp<br>
> +++ b/src/compiler/glsl_types.cpp<br>
> @@ -1086,7 +1086,7 @@ function_key_compare(const void *a, const void<br>
> *b)<br>
>     const glsl_type *const key2 = (glsl_type *) b;<br>
>  <br>
>     if (key1->length != key2->length)<br>
> -      return 1;<br>
> +      return false;<br>
>  <br>
>     return memcmp(key1->fields.parameters, key2->fields.parameters,<br>
>                   (key1->length + 1) * sizeof(*key1-<br>
> >fields.parameters)) == 0;<br>
</div></div></blockquote></div><br></div></div>