[07/13] target/openrisc: Fix lf.ftoi.s

Message ID	20190827000745.19645-8-richard.henderson@linaro.org
State	Superseded
Headers	show Delivered-To: patch@linaro.org Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; From: Richard Henderson <richard.henderson@linaro.org> To: qemu-devel@nongnu.org Date: Mon, 26 Aug 2019 17:07:39 -0700 Message-Id: <20190827000745.19645-8-richard.henderson@linaro.org> In-Reply-To: <20190827000745.19645-1-richard.henderson@linaro.org> References: <20190827000745.19645-1-richard.henderson@linaro.org> Subject: [Qemu-devel] [PATCH 07/13] target/openrisc: Fix lf.ftoi.s Precedence: list Cc: shorne@gmail.com Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" <qemu-devel-bounces+patch=linaro.org@nongnu.org>
Series	target/openrisc updates \| expand [00/13] target/openrisc updates [01/13] target/openrisc: Add DisasContext parameter to check_r0_write [02/13] target/openrisc: Replace cpu register array with a function [03/13] target/openrisc: Cache R0 in DisasContext [04/13] target/openrisc: Make VR and PPC read-only [05/13] target/openrisc: Move VR, UPR, DMMCFGR, IMMCFGR to cpu init [06/13] target/openrisc: Add VR2 and AVR special processor registers [07/13] target/openrisc: Fix lf.ftoi.s [08/13] target/openrisc: Check CPUCFG_OF32S for float insns [09/13] target/openrisc: Add support for ORFPX64A32 [10/13] target/openrisc: Implement unordered fp comparisons [11/13] target/openrisc: Implement move to/from FPCSR [12/13] target/openrisc: Implement l.adrp [13/13] target/openrisc: Update cpu "any" to v1.3

Message ID

20190827000745.19645-8-richard.henderson@linaro.org

State

Superseded

Headers

Received-SPF: pass (google.com: domain of
	qemu-devel-bounces+patch=linaro.org@nongnu.org designates
	209.51.188.17 as permitted sender) client-ip=209.51.188.17; 
From: Richard Henderson <richard.henderson@linaro.org>
To: qemu-devel@nongnu.org
Date: Mon, 26 Aug 2019 17:07:39 -0700
Message-Id: <20190827000745.19645-8-richard.henderson@linaro.org>
In-Reply-To: <20190827000745.19645-1-richard.henderson@linaro.org>
References: <20190827000745.19645-1-richard.henderson@linaro.org>
Subject: [Qemu-devel] [PATCH 07/13] target/openrisc: Fix lf.ftoi.s
Precedence: list
Cc: shorne@gmail.com
Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org
Sender: "Qemu-devel" <qemu-devel-bounces+patch=linaro.org@nongnu.org>

Series

target/openrisc updates | expand

The specification of this insn is round-to-zero. Signed-off-by: Richard Henderson <richard.henderson@linaro.org> --- target/openrisc/fpu_helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.17.1

Comments

Stafford Horne Aug. 27, 2019, 4:36 a.m. UTC | #1

On Mon, Aug 26, 2019 at 05:07:39PM -0700, Richard Henderson wrote:
> The specification of this insn is round-to-zero.

> 

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


Reviewed-by: Stafford Horne <shorne@gmail.com>


> ---

>  target/openrisc/fpu_helper.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/target/openrisc/fpu_helper.c b/target/openrisc/fpu_helper.c

> index b9d2ebbb8c..4cc5b297c5 100644

> --- a/target/openrisc/fpu_helper.c

> +++ b/target/openrisc/fpu_helper.c

> @@ -78,7 +78,7 @@ uint64_t HELPER(ftoid)(CPUOpenRISCState *env, uint64_t val)

>  

>  uint32_t HELPER(ftois)(CPUOpenRISCState *env, uint32_t val)

>  {

> -    return float32_to_int32(val, &env->fp_status);

> +    return float32_to_int32_round_to_zero(val, &env->fp_status);

>  }

>  

>  #define FLOAT_CALC(name)                                                  \

> -- 

> 2.17.1

>

diff --git a/target/openrisc/fpu_helper.c b/target/openrisc/fpu_helper.c
index b9d2ebbb8c..4cc5b297c5 100644
--- a/target/openrisc/fpu_helper.c
+++ b/target/openrisc/fpu_helper.c
@@ -78,7 +78,7 @@  uint64_t HELPER(ftoid)(CPUOpenRISCState *env, uint64_t val)
 
 uint32_t HELPER(ftois)(CPUOpenRISCState *env, uint32_t val)
 {
-    return float32_to_int32(val, &env->fp_status);
+    return float32_to_int32_round_to_zero(val, &env->fp_status);
 }
 
 #define FLOAT_CALC(name)                                                  \

[07/13] target/openrisc: Fix lf.ftoi.s

Commit Message

Comments

Patch