[Mesa-dev] [PATCH] tgsi: use truncf in micro_trunc

[Lars Hamre]

Gentle ping, I would appreciate a Gallium developer's eyes on this.

Regards,
Lars Hamre

On Thu, May 26, 2016 at 6:30 PM, Lars Hamre <chemecse at gmail.com> wrote:
> Switches to using truncf in micro_trunc.
>
> Fixes the following piglit tests (for softpipe):
>
> /spec/glsl-1.30/execution/built-in-functions/...
> fs-trunc-float
> fs-trunc-vec2
> fs-trunc-vec3
> fs-trunc-vec4
> vs-trunc-float
> vs-trunc-vec2
> vs-trunc-vec3
> vs-trunc-vec4
>
> /spec/glsl-1.50/execution/built-in-functions/...
> gs-trunc-float
> gs-trunc-vec2
> gs-trunc-vec3
> gs-trunc-vec4
>
> Signed-off-by: Lars Hamre <chemecse at gmail.com>
>
> ---
>
> NOTE: someone with access will need to commit this post
>       review process
>
>  src/gallium/auxiliary/tgsi/tgsi_exec.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/gallium/auxiliary/tgsi/tgsi_exec.c b/src/gallium/auxiliary/tgsi/tgsi_exec.c
> index baf4a89..2ae1482 100644
> --- a/src/gallium/auxiliary/tgsi/tgsi_exec.c
> +++ b/src/gallium/auxiliary/tgsi/tgsi_exec.c
> @@ -676,10 +676,10 @@ static void
>  micro_trunc(union tgsi_exec_channel *dst,
>              const union tgsi_exec_channel *src)
>  {
> -   dst->f[0] = (float)(int)src->f[0];
> -   dst->f[1] = (float)(int)src->f[1];
> -   dst->f[2] = (float)(int)src->f[2];
> -   dst->f[3] = (float)(int)src->f[3];
> +   dst->f[0] = truncf(src->f[0]);
> +   dst->f[1] = truncf(src->f[1]);
> +   dst->f[2] = truncf(src->f[2]);
> +   dst->f[3] = truncf(src->f[3]);
>  }
>
>  static void
> --
> 2.5.5
>