qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] target/m68k: Change fpu_rom from const static a


From: Peter Maydell
Subject: Re: [Qemu-devel] [PATCH] target/m68k: Change fpu_rom from const static array to switch
Date: Mon, 4 Sep 2017 16:02:07 +0100

On 4 September 2017 at 15:09, Laurent Vivier <address@hidden> wrote:
> You can:
>
> either replace the "#define floatx80_pi make_floatx80(...)" by a "const
> floatx80 floatx80_pi = make_floatx80_init(...)"
>
> or replace all the macros in the m68k/fpu_helper.c array by
> make_floatx80_init(...)

Taking a step back, what's different about floatx80 and float12
that means they need separate _init and non-init versions of
the macros, when for float16/float32/float64 we instead have
#define make_float32(x) __extension__ ({ float32 f32_val = {x}; f32_val; })
#define const_float32(x) { x }

? Could we move to consistency for the macro naming we're using?

thanks
-- PMM



reply via email to

[Prev in Thread] Current Thread [Next in Thread]