[Mesa-dev] [PATCH mesa] radv: s/abs/fabsf/ for floats

[Bas Nieuwenhuizen]

Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>

On Thu, Oct 18, 2018 at 5:46 PM Eric Engestrom <eric.engestrom at intel.com> wrote:
>
> Fixes: a4c4efad89eceb26cf82 "radv: Rework guard band calculation"
> Cc: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
> Signed-off-by: Eric Engestrom <eric.engestrom at intel.com>
> ---
>  src/amd/vulkan/si_cmd_buffer.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c
> index de057657ee70d354e910..52daf99414790d4764b6 100644
> --- a/src/amd/vulkan/si_cmd_buffer.c
> +++ b/src/amd/vulkan/si_cmd_buffer.c
> @@ -516,16 +516,16 @@ si_write_scissors(struct radeon_cmdbuf *cs, int first,
>                 VkRect2D scissor = si_intersect_scissor(&scissors[i], &viewport_scissor);
>
>                 get_viewport_xform(viewports + i, scale, translate);
> -               scale[0] = abs(scale[0]);
> -               scale[1] = abs(scale[1]);
> +               scale[0] = fabsf(scale[0]);
> +               scale[1] = fabsf(scale[1]);
>
>                 if (scale[0] < 0.5)
>                         scale[0] = 0.5;
>                 if (scale[1] < 0.5)
>                         scale[1] = 0.5;
>
> -               guardband_x = MIN2(guardband_x, (max_range - abs(translate[0])) / scale[0]);
> -               guardband_y = MIN2(guardband_y, (max_range - abs(translate[1])) / scale[1]);
> +               guardband_x = MIN2(guardband_x, (max_range - fabsf(translate[0])) / scale[0]);
> +               guardband_y = MIN2(guardband_y, (max_range - fabsf(translate[1])) / scale[1]);
>
>                 radeon_emit(cs, S_028250_TL_X(scissor.offset.x) |
>                             S_028250_TL_Y(scissor.offset.y) |
> --
> Cheers,
>   Eric
>