<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 26, 2015 at 3:33 PM, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>></span> wrote:<br><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 01/26/2015 02:22 PM, Jason Ekstrand wrote:<br>
> Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=88806" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=88806</a><br>
> ---<br>
>  src/glsl/nir/nir_opcodes.py | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/src/glsl/nir/nir_opcodes.py b/src/glsl/nir/nir_opcodes.py<br>
> index f54a017..3f21539 100644<br>
> --- a/src/glsl/nir/nir_opcodes.py<br>
> +++ b/src/glsl/nir/nir_opcodes.py<br>
> @@ -480,7 +480,7 @@ else<br>
>  opcode("ldexp", 0, tunsigned, [0, 0], [tfloat, tint], "", """<br>
>  dst = ldexp(src0, src1);<br>
>  /* flush denormals to zero. */<br>
> -if (!isnormal(dst))<br>
> +if (fpclassify(dst) != FP_NORMAL)<br>
<br>
</div></div>GCC documentation claims that isnormal is faster than fpclassify(x) ==<br>
FP_NORMAL.  Do we care?<br>
<br>
If we care, we should provide isnormal() in macros.h.<br></blockquote><div><br></div><div>For the moment that sounds like too much work.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
If not,<br>
<br>
Reviewed-by: Ian Romanick <<a href="mailto:ian.d.romanick@intel.com">ian.d.romanick@intel.com</a>><br></blockquote><div><br></div><div>Thanks,<br></div><div>--Jason<br> <br></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>
>     dst = copysign(0.0f, src0);<br>
>  """)<br>
><br>
><br>
<br>
</div></div></blockquote></div><br></div></div>