[v2] fpu: Fix OCP FP8 E4M3 conversion issues

[PATCH v2 0/2] fpu: Fix OCP FP8 E4M3 conversion issues

Max Chou posted 2 patches 1 month, 2 weeks ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260226071817.1417875-1-max.chou@sifive.com

Maintainers: Aurelien Jarno <aurelien@aurel32.net>, Peter Maydell <peter.maydell@linaro.org>, "Alex Bennée" <alex.bennee@linaro.org>

fpu/softfloat-parts.c.inc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

Expand all Fold all

[PATCH v2 0/2] fpu: Fix OCP FP8 E4M3 conversion issues

Posted by Max Chou 1 month, 2 weeks ago

This series fix following issues related to OCP FP8 E4M3 conversion.

- Numeric overflow to E4M3 produces incorrect exponent values due to a
  missing local variable update after calling uncanon_e4m3_overflow.
- Converting infinity to E4M3 incorrectly raises OF|NX exception flags.

Changes in version 2:
- The “fpu: Simplify OCP FP8 E4M3 NaN classification in parts_canonicalize”
  patch has been removed. This change retains the flexibility to classify NaNs.

rnax

Max Chou (2):
  fpu: Fix repacking issues in the uncanonical step for E4M3 overflow
  fpu: Fix unexpected exception flags when converting infinity to OCP
    E4M3

 fpu/softfloat-parts.c.inc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

-- 
2.52.0

Re: [PATCH v2 0/2] fpu: Fix OCP FP8 E4M3 conversion issues

Posted by Alistair Francis 1 month, 1 week ago

On Thu, Feb 26, 2026 at 5:19 PM Max Chou <max.chou@sifive.com> wrote:
>
> This series fix following issues related to OCP FP8 E4M3 conversion.
>
> - Numeric overflow to E4M3 produces incorrect exponent values due to a
>   missing local variable update after calling uncanon_e4m3_overflow.
> - Converting infinity to E4M3 incorrectly raises OF|NX exception flags.
>
> Changes in version 2:
> - The “fpu: Simplify OCP FP8 E4M3 NaN classification in parts_canonicalize”
>   patch has been removed. This change retains the flexibility to classify NaNs.
>
> rnax
>
> Max Chou (2):
>   fpu: Fix repacking issues in the uncanonical step for E4M3 overflow
>   fpu: Fix unexpected exception flags when converting infinity to OCP
>     E4M3

Thanks!

Applied to riscv-to-apply.next

Alistair

>
>  fpu/softfloat-parts.c.inc | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> --
> 2.52.0
>
>