Re: [musl] [PATCH 1/2] math: add riscv64 llround/llroundf

[Rich Felker <dalias@libc.org>]

On Wed, Jun 12, 2024 at 05:00:17PM +0800, Meng Zhuo wrote:
> ---
>  src/math/riscv64/llround.c  | 16 ++++++++++++++++
>  src/math/riscv64/llroundf.c | 16 ++++++++++++++++
>  2 files changed, 32 insertions(+)
>  create mode 100644 src/math/riscv64/llround.c
>  create mode 100644 src/math/riscv64/llroundf.c
> 
> diff --git a/src/math/riscv64/llround.c b/src/math/riscv64/llround.c
> new file mode 100644
> index 00000000..fa646b5a
> --- /dev/null
> +++ b/src/math/riscv64/llround.c
> @@ -0,0 +1,16 @@
> +#include <math.h>
> +
> +#if __riscv_flen >= 64
> +
> +long long llround(double x)
> +{
> +	long long n;
> +	__asm__ ("fcvt.l.d %0, %1, rmm" : "=r"(n) : "f"(x));
> +	return n;
> +}
> +
> +#else
> +
> +#include "../llround.c"
> +
> +#endif
> diff --git a/src/math/riscv64/llroundf.c b/src/math/riscv64/llroundf.c
> new file mode 100644
> index 00000000..db2d58df
> --- /dev/null
> +++ b/src/math/riscv64/llroundf.c
> @@ -0,0 +1,16 @@
> +#include <math.h>
> +
> +#if __riscv_flen >= 32
> +
> +long long llroundf(float x)
> +{
> +	long long n;
> +	__asm__ ("fcvt.l.s %0, %1, rmm" : "=r"(n) : "f"(x));
> +	return n;
> +}
> +
> +#else
> +
> +#include "../llroundf.c"
> +
> +#endif
> -- 
> 2.39.2

Is there a reason you only did llround[f] and not lround[f]?

Rich