Patchwork [4/4] target-arm: Use the standard FPSCR value for VRSQRTS

login
register
mail settings
Submitter Peter Maydell
Date Jan. 7, 2011, 9:26 a.m.
Message ID <1294392387-24300-5-git-send-email-peter.maydell@linaro.org>
Download mbox | patch
Permalink /patch/77862/
State New
Headers show

Comments

Peter Maydell - Jan. 7, 2011, 9:26 a.m.
VSQRTS always uses the standard FPSCR value as it is a Neon instruction.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 target-arm/helper.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Aurelien Jarno - Jan. 10, 2011, 7:20 p.m.
On Fri, Jan 07, 2011 at 09:26:27AM +0000, Peter Maydell wrote:
> VSQRTS always uses the standard FPSCR value as it is a Neon instruction.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  target-arm/helper.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Reviewed-by: Aurelien Jarno <aurelien@aurel32.net>
 
> diff --git a/target-arm/helper.c b/target-arm/helper.c
> index d779055..8186500 100644
> --- a/target-arm/helper.c
> +++ b/target-arm/helper.c
> @@ -2616,7 +2616,7 @@ float32 HELPER(recps_f32)(float32 a, float32 b, CPUState *env)
>  
>  float32 HELPER(rsqrts_f32)(float32 a, float32 b, CPUState *env)
>  {
> -    float_status *s = &env->vfp.fp_status;
> +    float_status *s = &env->vfp.standard_fp_status;
>      float32 two = int32_to_float32(2, s);
>      float32 three = int32_to_float32(3, s);
>      float32 product;
> -- 
> 1.6.3.3
> 
> 
>

Patch

diff --git a/target-arm/helper.c b/target-arm/helper.c
index d779055..8186500 100644
--- a/target-arm/helper.c
+++ b/target-arm/helper.c
@@ -2616,7 +2616,7 @@  float32 HELPER(recps_f32)(float32 a, float32 b, CPUState *env)
 
 float32 HELPER(rsqrts_f32)(float32 a, float32 b, CPUState *env)
 {
-    float_status *s = &env->vfp.fp_status;
+    float_status *s = &env->vfp.standard_fp_status;
     float32 two = int32_to_float32(2, s);
     float32 three = int32_to_float32(3, s);
     float32 product;