[Mesa-dev] [PATCH 2/4] mesa: Replace uses of IROUND{, D, 64} with libm functions.

Brian Paul brianp at vmware.com
Thu May 26 20:47:29 UTC 2016


On 05/26/2016 12:28 PM, Matt Turner wrote:
> IROUND   is replaces with lroundf.
> IROUNDD  is replaces with lround.
> IROUND64 is replaced with llroundf.
> ---
> This is a resend of a patch from last year.
>
> Roland suggested using rint instead of round since it's cheaper, but it
> actually causes test failures in some dEQP tests. Additionally, while
> rint is cheaper than round, it's not as bad as he thought since we're
> using -fno-math-errno -fno-trapping-math.
>
> Brian said it would be interesting to run the old conformance suite
> with swrast. I have not done that... I don't think I know where that
> conformance suite is.

I just tested conform on swrast with this patch series.  No regressions.

Tested-by: Brian Paul <brianp at vmware.com>



More information about the mesa-dev mailing list