<div dir="ltr">With the version cutoff fixed, this and the patch it builds on are (squashed together or not):<div><br></div><div>Reviewed-by: Chris Forbes <<a href="mailto:chrisforbes@google.com">chrisforbes@google.com</a>></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 13, 2016 at 4:58 PM, Ilia Mirkin <span dir="ltr"><<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</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 Fri, May 13, 2016 at 12:51 AM, Dave Airlie <<a href="mailto:airlied@gmail.com">airlied@gmail.com</a>> wrote:<br>
>>> second argument is for ES... 0 means "never").<br>
>><br>
>> I see.  (You can tell how much of this sort of code I've written...).<br>
>><br>
>> I don't know that I'd trust me but it looks fine as far add I can see.<br>
>> Thanks for taking care of 4.50 while you were in the neighborhood.  For what<br>
>> it's worth,<br>
><br>
> Do we know if the addition of swizzles and just correcting an oversight?<br>
><br>
> Should we just enable it unconditionally.<br>
<br>
</span>Good question.<br>
<br>
Looks like the cut-off is actually 4.40, not 4.50 (oops). I don't see<br>
it explicitly listed in changes of the 4.40 spec.<br>
<br>
In 4.30: "For all of the interpolation functions, interpolant must be<br>
an input variable or an element of an input<br>
variable declared as an array. Component selection operators (e.g.,<br>
.xy) may not be used when specifying<br>
interpolant."<br>
<br>
In 4.40: "For all of the interpolation functions, interpolant must be<br>
an input variable or an element of an input<br>
variable declared as an array. Component selection operators (e.g.,<br>
.xy) may be used when specifying<br>
interpolant."<br>
<br>
I'd say it's pretty clear that it's not allowed in 4.30 and allowed in<br>
4.40. I'm going to fix the version cut-off locally, but if you want me<br>
to enable it everywhere, let me know.<br>
<span class="HOEnZb"><font color="#888888"><br>
  -ilia<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">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>
</div></div></blockquote></div><br></div>