Reviewed-by: Connor Abbott <cwabbo...@gmail.com>
On Mon, Jan 26, 2015 at 12:43 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote:
> Previously, we called the abs() function in math.h. However, this involves
> unnecessarily going through double. This commit changes it to use integers
> directly with a ternary.
> ---
> src/glsl/nir/nir_opcodes.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/glsl/nir/nir_opcodes.py b/src/glsl/nir/nir_opcodes.py
> index 0fa8bf5..f54a017 100644
> --- a/src/glsl/nir/nir_opcodes.py
> +++ b/src/glsl/nir/nir_opcodes.py
> @@ -147,7 +147,7 @@ unop("inot", tint, "~src0") # invert every bit of the
> integer
> unop("fnot", tfloat, "(src0 == 0.0f) ? 1.0f : 0.0f")
> unop("fsign", tfloat, "(src0 == 0.0f) ? 0.0f : ((src0 > 0.0f) ? 1.0f :
> -1.0f)")
> unop("isign", tint, "(src0 == 0) ? 0 : ((src0 > 0) ? 1 : -1)")
> -unop("iabs", tint, "abs(src0)")
> +unop("iabs", tint, "(src0 < 0) ? -src0 : src0")
> unop("fabs", tfloat, "fabsf(src0)")
> unop("fsat", tfloat, "(src0 > 1.0f) ? 1.0f : ((src0 <= 0.0f) ? 0.0f : src0)")
> unop("frcp", tfloat, "1.0f / src0")
> --
> 2.2.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
