<br><br><div class="gmail_quote"><div dir="ltr">On Fri, May 13, 2016, 7:49 PM Tom Stellard via Libclc-dev <<a href="mailto:libclc-dev@lists.llvm.org">libclc-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, May 13, 2016 at 08:36:18PM -0400, Jan Vesely via Libclc-dev wrote:<br>
> Fixes fdim piglit on Turks<br>
><br>
> Signed-off-by: Jan Vesely <<a href="mailto:jan.vesely@rutgers.edu" target="_blank">jan.vesely@rutgers.edu</a>><br>
> ---<br>
><br>
> Not sure when llvm started inserting double promotion:<br>
> <unknown>:0:0: in function test_1_fdim_float void (float addrspace(1)*, float addrspace(1)*, float addrspace(1)*): unsupported call to function __extendsfdf2<br>
><br>
>  generic/lib/math/fdim.inc | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/generic/lib/math/fdim.inc b/generic/lib/math/fdim.inc<br>
> index a67c76e..0e57513 100644<br>
> --- a/generic/lib/math/fdim.inc<br>
> +++ b/generic/lib/math/fdim.inc<br>
> @@ -25,7 +25,7 @@<br>
>  _CLC_OVERLOAD _CLC_DEF __CLC_GENTYPE fdim(__CLC_GENTYPE x, __CLC_GENTYPE y) {<br>
>      if (__builtin_isnan(x) || __builtin_isnan(y))<br>
>          return as_float(QNANBITPATT_SP32);<br>
> -    return __builtin_fmax(x - y, 0);<br>
> +    return __builtin_fmaxf(x - y, 0);<br>
<br>
Is there a reason we aren't just calling the OpenCL builtin fmax ?<br></blockquote></div><div><br></div><div>Reason? Or good reason?  </div><div><br></div><div>If we change this to either __builtin_fmaxf or fmax (I'd go with that), should we also make that an explicit 0.0f?</div><div><br></div><div>--Aaron</div><div><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-Tom<br>
<br>
>  }<br>
>  #define __CLC_FDIM_VEC(width) \<br>
>  _CLC_OVERLOAD _CLC_DEF float##width fdim(float##width x, float##width y) { \<br>
> --<br>
> 2.5.5<br>
><br>
> _______________________________________________<br>
> Libclc-dev mailing list<br>
> <a href="mailto:Libclc-dev@lists.llvm.org" target="_blank">Libclc-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/libclc-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/libclc-dev</a><br>
_______________________________________________<br>
Libclc-dev mailing list<br>
<a href="mailto:Libclc-dev@lists.llvm.org" target="_blank">Libclc-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/libclc-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/libclc-dev</a><br>
</blockquote></div>