target/arm: Implement FEAT_AFP and FEAT_RPRES

[PATCH 08/76] target/arm: Use fp_status_a32 in vjvct helper

Posted by Peter Maydell 2 months, 1 week ago

Use fp_status_a32 in the vjcvt helper function; this is called only
from the A32/T32 decoder and is not used inside a
set_rmode/restore_rmode sequence.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 target/arm/vfp_helper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target/arm/vfp_helper.c b/target/arm/vfp_helper.c
index 7475f97e0ce..0671ba3a88b 100644
--- a/target/arm/vfp_helper.c
+++ b/target/arm/vfp_helper.c
@@ -1144,7 +1144,7 @@ uint64_t HELPER(fjcvtzs)(float64 value, float_status *status)
 
 uint32_t HELPER(vjcvt)(float64 value, CPUARMState *env)
 {
-    uint64_t pair = HELPER(fjcvtzs)(value, &env->vfp.fp_status);
+    uint64_t pair = HELPER(fjcvtzs)(value, &env->vfp.fp_status_a32);
     uint32_t result = pair;
     uint32_t z = (pair >> 32) == 0;
 
-- 
2.34.1

Re: [PATCH 08/76] target/arm: Use fp_status_a32 in vjvct helper

Posted by Richard Henderson 2 months, 1 week ago

On 1/24/25 08:27, Peter Maydell wrote:
> Use fp_status_a32 in the vjcvt helper function; this is called only
> from the A32/T32 decoder and is not used inside a
> set_rmode/restore_rmode sequence.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>   target/arm/vfp_helper.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/target/arm/vfp_helper.c b/target/arm/vfp_helper.c
> index 7475f97e0ce..0671ba3a88b 100644
> --- a/target/arm/vfp_helper.c
> +++ b/target/arm/vfp_helper.c
> @@ -1144,7 +1144,7 @@ uint64_t HELPER(fjcvtzs)(float64 value, float_status *status)
>   
>   uint32_t HELPER(vjcvt)(float64 value, CPUARMState *env)
>   {
> -    uint64_t pair = HELPER(fjcvtzs)(value, &env->vfp.fp_status);
> +    uint64_t pair = HELPER(fjcvtzs)(value, &env->vfp.fp_status_a32);
>       uint32_t result = pair;
>       uint32_t z = (pair >> 32) == 0;
>   

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~