[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 12/13] target/riscv: rationalise softfloat in
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v3 12/13] target/riscv: rationalise softfloat includes |
Date: |
Tue, 13 Aug 2019 15:05:44 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 8/13/19 2:49 PM, Alex Bennée wrote:
> We should avoid including the whole of softfloat headers in cpu.h and
> explicitly include it only where we will be calling softfloat
> functions. We can use the -types.h and -helpers.h in cpu.h for the few
> bits that are global.
>
> Signed-off-by: Alex Bennée <address@hidden>
> Reviewed-by: Richard Henderson <address@hidden>
> Reviewed-by: Alistair Francis <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> target/riscv/cpu.c | 1 +
> target/riscv/cpu.h | 2 +-
> target/riscv/fpu_helper.c | 1 +
> 3 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c
> index f8d07bd20ad..6d52f97d7c3 100644
> --- a/target/riscv/cpu.c
> +++ b/target/riscv/cpu.c
> @@ -27,6 +27,7 @@
> #include "qemu/error-report.h"
> #include "hw/qdev-properties.h"
> #include "migration/vmstate.h"
> +#include "fpu/softfloat-helpers.h"
>
> /* RISC-V CPU definitions */
>
> diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
> index 0adb307f329..240b31e2ebb 100644
> --- a/target/riscv/cpu.h
> +++ b/target/riscv/cpu.h
> @@ -22,7 +22,7 @@
>
> #include "qom/cpu.h"
> #include "exec/cpu-defs.h"
> -#include "fpu/softfloat.h"
> +#include "fpu/softfloat-types.h"
>
> #define TCG_GUEST_DEFAULT_MO 0
>
> diff --git a/target/riscv/fpu_helper.c b/target/riscv/fpu_helper.c
> index b4f818a6465..0b79562a690 100644
> --- a/target/riscv/fpu_helper.c
> +++ b/target/riscv/fpu_helper.c
> @@ -21,6 +21,7 @@
> #include "qemu/host-utils.h"
> #include "exec/exec-all.h"
> #include "exec/helper-proto.h"
> +#include "fpu/softfloat.h"
>
> target_ulong riscv_cpu_get_fflags(CPURISCVState *env)
> {
>
- Re: [Qemu-devel] [PATCH v3 05/13] fpu: replace LIT64 with UINT64_C macros, (continued)
- [Qemu-devel] [PATCH v3 06/13] target/m68k: replace LIT64 with UINT64_C macros, Alex Bennée, 2019/08/13
- [Qemu-devel] [PATCH v3 09/13] fpu: make softfloat-macros "self-contained", Alex Bennée, 2019/08/13
- [Qemu-devel] [PATCH v3 10/13] fpu: rename softfloat-specialize.h -> .inc.c, Alex Bennée, 2019/08/13
- [Qemu-devel] [PATCH v3 08/13] fpu: move inline helpers into a separate header, Alex Bennée, 2019/08/13
- [Qemu-devel] [PATCH v3 11/13] target/mips: rationalise softfloat includes, Alex Bennée, 2019/08/13
- [Qemu-devel] [PATCH v3 13/13] targets (various): use softfloat-helpers.h where we can, Alex Bennée, 2019/08/13
- [Qemu-devel] [PATCH v3 12/13] target/riscv: rationalise softfloat includes, Alex Bennée, 2019/08/13
- Re: [Qemu-devel] [PATCH v3 12/13] target/riscv: rationalise softfloat includes,
Philippe Mathieu-Daudé <=
- Re: [Qemu-devel] [PATCH v3 00/13] softfloat updates (include tweaks, rm LIT64), no-reply, 2019/08/13
- Re: [Qemu-devel] [PATCH v3 00/13] softfloat updates (include tweaks, rm LIT64), no-reply, 2019/08/13
- Re: [Qemu-devel] [PATCH v3 00/13] softfloat updates (include tweaks, rm LIT64), Markus Armbruster, 2019/08/13