diff mbox series

[PULL,22/32] target/riscv: Fix froundnx.h nanbox check

Message ID	20240627100053.150937-23-alistair.francis@wdc.com
State	New
Headers	show Return-Path: <qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org> From: Alistair Francis <alistair23@gmail.com> To: qemu-devel@nongnu.org Cc: alistair23@gmail.com, Branislav Brzak <brzakbranislav@gmail.com>, Alistair Francis <alistair.francis@wdc.com>, Richard Henderson <richard.henderson@linaro.org> Subject: [PULL 22/32] target/riscv: Fix froundnx.h nanbox check Date: Thu, 27 Jun 2024 20:00:43 +1000 Message-ID: <20240627100053.150937-23-alistair.francis@wdc.com> In-Reply-To: <20240627100053.150937-1-alistair.francis@wdc.com> References: <20240627100053.150937-1-alistair.francis@wdc.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=2607:f8b0:4864:20::233; envelope-from=alistair23@gmail.com; helo=mail-oi1-x233.google.com X-Spam_score_int: -17 X-Spam_score: -1.8 X-Spam_bar: - X-Spam_report: (-1.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action Precedence: list Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org
Series	[PULL,01/32] target/riscv: Extend virtual irq csrs masks to be 64 bit wide. \| expand [PULL,01/32] target/riscv: Extend virtual irq csrs masks to be 64 bit wide. [PULL,02/32] target/riscv: Move Guest irqs out of the core local irqs range. [PULL,03/32] target/riscv: zvbb implies zvkb [PULL,04/32] hw/riscv/virt.c: add address-cells in create_fdt_one_aplic() [PULL,05/32] hw/riscv/virt.c: add aplic nodename helper [PULL,06/32] hw/riscv/virt.c: rename aplic nodename to 'interrupt-controller' [PULL,07/32] hw/riscv/virt.c: aplic DT: add 'qemu, aplic' to 'compatible' [PULL,08/32] hw/riscv/virt.c: aplic DT: rename prop to 'riscv, delegation' [PULL,09/32] hw/riscv/virt.c: change imsic nodename to 'interrupt-controller' [PULL,10/32] hw/riscv/virt.c: imsics DT: add 'qemu, imsics' to 'compatible' [PULL,11/32] hw/riscv/virt.c: imsics DT: add '#msi-cells' [PULL,12/32] target/riscv/kvm: add software breakpoints support [PULL,13/32] target/riscv/kvm: handle the exit with debug reason [PULL,14/32] target/riscv/kvm: define TARGET_KVM_HAVE_GUEST_DEBUG [PULL,15/32] target/riscv: Reuse the conversion function of priv_spec [PULL,16/32] target/riscv: Define macros and variables for ss1p13 [PULL,17/32] target/riscv: Add 'P1P13' bit in SMSTATEEN0 [PULL,18/32] target/riscv: Add MEDELEGH, HEDELEGH csrs for RV32 [PULL,19/32] target/riscv: Reserve exception codes for sw-check and hw-err [PULL,20/32] target/riscv: Support the version for ss1p13 [PULL,21/32] hw/riscv/virt.c: Make block devices default to virtio [PULL,22/32] target/riscv: Fix froundnx.h nanbox check [PULL,23/32] target/riscv: fix instructions count handling in icount mode [PULL,24/32] target/riscv: Introduce extension implied rules definition [PULL,25/32] target/riscv: Introduce extension implied rule helpers [PULL,26/32] target/riscv: Add MISA extension implied rules [PULL,27/32] target/riscv: Add multi extension implied rules [PULL,28/32] target/riscv: Add Zc extension implied rule [PULL,29/32] target/riscv: Remove extension auto-update check statements [PULL,30/32] target/riscv: Add functions for common matching conditions of trigger [PULL,31/32] target/riscv: Apply modularized matching conditions for watchpoint [PULL,32/32] target/riscv: Apply modularized matching conditions for icount trigger

Commit Message

Alistair Francis June 27, 2024, 10 a.m. UTC

From: Branislav Brzak <brzakbranislav@gmail.com>

helper_froundnx_h function mistakenly uses single percision nanbox
check instead of the half percision one. This patch fixes the issue.

Signed-off-by: Branislav Brzak <brzakbranislav@gmail.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-ID: <20240608214546.226963-1-brzakbranislav@gmail.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
---
 target/riscv/fpu_helper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff mbox series

Patch

diff --git a/target/riscv/fpu_helper.c b/target/riscv/fpu_helper.c
index 871a70a316..91b1a56d10 100644
--- a/target/riscv/fpu_helper.c
+++ b/target/riscv/fpu_helper.c
@@ -676,7 +676,7 @@  uint64_t helper_fround_h(CPURISCVState *env, uint64_t rs1)
 
 uint64_t helper_froundnx_h(CPURISCVState *env, uint64_t rs1)
 {
-    float16 frs1 = check_nanbox_s(env, rs1);
+    float16 frs1 = check_nanbox_h(env, rs1);
     frs1 = float16_round_to_int(frs1, &env->fp_status);
     return nanbox_h(env, frs1);
 }