Re: [PATCH] riscv: Fix fenv.h support

Jeff Johnston <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <CAOox84tRYcmgQUpoyAD7NsqE3u1HrFjpqWdYq6cVMpbmToagww@mail.gmail.com>
Feel free to merge.  The file already mentions __riscv_flen in the #else
clause.

-- Jeff J.

On Thu, Oct 12, 2023 at 6:06 AM Sebastian Huber <
[email protected]> wrote:

> Use the same C preprocessor expressions to define FE_RMODE_MASK and
> __RISCV_HARD_FLOAT.
>
> The problem was noticed on GCC 10 which does not define __riscv_f.
> ---
>  newlib/libc/machine/riscv/sys/fenv.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/newlib/libc/machine/riscv/sys/fenv.h
> b/newlib/libc/machine/riscv/sys/fenv.h
> index 1d577d527b..98387d2205 100644
> --- a/newlib/libc/machine/riscv/sys/fenv.h
> +++ b/newlib/libc/machine/riscv/sys/fenv.h
> @@ -14,7 +14,7 @@
>
>  #include <stddef.h>
>
> -#if defined(__riscv_f) || defined(__riscv_zfinx)
> +#if defined(__riscv_flen) || defined(__riscv_zfinx)
>
>  /* Per "The RISC-V Instruction Set Manual: Volume I: User-Level ISA:
>   * Version 2.1", Section 8.2, "Floating-Point Control and Status
> --
> 2.35.3
>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.