Stream: git-wasmtime

Topic: wasmtime / issue #7322 cranelift: Disable NaN Canonicaliz...

Wasmtime GitHub notifications bot (Oct 21 2023 at 23:50):

afonso360 added the cranelift:area:riscv64 label to Issue #7322.

Wasmtime GitHub notifications bot (Oct 21 2023 at 23:50):

afonso360 opened issue #7322:

:wave: Hey,

In RISC-V almost all floating point instructions return a canonical NaN when its result would be a NaN. In cranelift we have a separate pass that includes some NaN normalization code when the enable_nan_canonicalization flag is enabled. RISC-V's canonical NaN pattern is also defined to be the same as WASM's canonical NaN.

This isn't really required for RISC-V and just causes additional overhead, so it would be nice if we could disable it. This thought occurred to me while reading #7321 which might make that easier by querying each backend and applying those legalization rules conditionally.

Last updated: Nov 22 2024 at 16:03 UTC