[v2,for-10.0,44/54] target/openrisc: Set default NaN pattern explicitly

Commit Message

Peter Maydell Dec. 2, 2024, 1:13 p.m. UTC

Set the default NaN pattern explicitly for openrisc.

Signed-off-by: Peter Maydell <[e-mail]>
---
 target/openrisc/cpu.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Richard Henderson Dec. 2, 2024, 5:31 p.m. UTC | #1

On 12/2/24 07:13, Peter Maydell wrote:
> Set the default NaN pattern explicitly for openrisc.
> 
> Signed-off-by: Peter Maydell <[e-mail]>
> ---
>   target/openrisc/cpu.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/target/openrisc/cpu.c b/target/openrisc/cpu.c
> index b96561d1f26..3ccf85e95f0 100644
> --- a/target/openrisc/cpu.c
> +++ b/target/openrisc/cpu.c
> @@ -111,6 +111,8 @@ static void openrisc_cpu_reset_hold(Object *obj, ResetType type)
>        */
>       set_float_2nan_prop_rule(float_2nan_prop_x87, &cpu->env.fp_status);
>   
> +    /* Default NaN: sign bit clear, frac msb set */
> +    set_float_default_nan_pattern(0b01000000, &cpu->env.fp_status);
>   
>   #ifndef CONFIG_USER_ONLY
>       cpu->env.picmr = 0x00000000;

Reviewed-by: Richard Henderson <[e-mail]>

r~

Patch

diff --git a/target/openrisc/cpu.c b/target/openrisc/cpu.c
index b96561d1f26..3ccf85e95f0 100644
--- a/target/openrisc/cpu.c
+++ b/target/openrisc/cpu.c
@@ -111,6 +111,8 @@  static void openrisc_cpu_reset_hold(Object *obj, ResetType type)
      */
     set_float_2nan_prop_rule(float_2nan_prop_x87, &cpu->env.fp_status);
 
+    /* Default NaN: sign bit clear, frac msb set */
+    set_float_default_nan_pattern(0b01000000, &cpu->env.fp_status);
 
 #ifndef CONFIG_USER_ONLY
     cpu->env.picmr = 0x00000000;