Re: [Qemu-devel] [PATCH 3/6] target/arm: Use fp_status_fp16 for do_fmpa_zpzzz_h

2018-08-11 Thread Laurent Desnogues
On Fri, Aug 10, 2018 at 9:31 PM, Richard Henderson
 wrote:
> This makes float16_muladd correctly use FZ16 not FZ.
>
> Fixes: 6ceabaad110
> Cc: qemu-sta...@nongnu.org (3.0.1)
> Reported-by: Laurent Desnogues 
> Signed-off-by: Richard Henderson 

Tested-by: Laurent Desnogues 
Reviewed-by: Laurent Desnogues 

Laurent

> ---
>  target/arm/sve_helper.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/arm/sve_helper.c b/target/arm/sve_helper.c
> index 5bae600d17..f8434e68f2 100644
> --- a/target/arm/sve_helper.c
> +++ b/target/arm/sve_helper.c
> @@ -3398,7 +3398,7 @@ static void do_fmla_zpzzz_h(CPUARMState *env, void *vg, 
> uint32_t desc,
>  e1 = *(uint16_t *)(vn + H1_2(i)) ^ neg1;
>  e2 = *(uint16_t *)(vm + H1_2(i));
>  e3 = *(uint16_t *)(va + H1_2(i)) ^ neg3;
> -r = float16_muladd(e1, e2, e3, 0, >vfp.fp_status);
> +r = float16_muladd(e1, e2, e3, 0, >vfp.fp_status_f16);
>  *(uint16_t *)(vd + H1_2(i)) = r;
>  }
>  } while (i & 63);
> --
> 2.17.1
>



[Qemu-devel] [PATCH 3/6] target/arm: Use fp_status_fp16 for do_fmpa_zpzzz_h

2018-08-10 Thread Richard Henderson
This makes float16_muladd correctly use FZ16 not FZ.

Fixes: 6ceabaad110
Cc: qemu-sta...@nongnu.org (3.0.1)
Reported-by: Laurent Desnogues 
Signed-off-by: Richard Henderson 
---
 target/arm/sve_helper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target/arm/sve_helper.c b/target/arm/sve_helper.c
index 5bae600d17..f8434e68f2 100644
--- a/target/arm/sve_helper.c
+++ b/target/arm/sve_helper.c
@@ -3398,7 +3398,7 @@ static void do_fmla_zpzzz_h(CPUARMState *env, void *vg, 
uint32_t desc,
 e1 = *(uint16_t *)(vn + H1_2(i)) ^ neg1;
 e2 = *(uint16_t *)(vm + H1_2(i));
 e3 = *(uint16_t *)(va + H1_2(i)) ^ neg3;
-r = float16_muladd(e1, e2, e3, 0, >vfp.fp_status);
+r = float16_muladd(e1, e2, e3, 0, >vfp.fp_status_f16);
 *(uint16_t *)(vd + H1_2(i)) = r;
 }
 } while (i & 63);
-- 
2.17.1