On Mon, 30 Oct 2023, Robin Dapp wrote: > Hi, > > this patch adds expanders for fmin and fmax and the associated > cond and reduc ones. As per RISC-V V spec 1.0 vfmin/vfmax are > IEEE 754-2008 compliant so that should be ok.
Aren't they actually the IEEE 754-2019 operations (with different signaling NaN semantics; C functions such as fmaximum in C23), not the IEEE 754-2008 operations (C functions such as fmax)? V spec 1.0 says "The vector floating-point vfmin and vfmax instructions have the same behavior as the corresponding scalar floating-point instructions in version 2.2 of the RISC-V F/D/Q extension.". And version 2.2 of F/D/Q (which is *not* version 2.2 of the instruction set, it's later than that) changed the scalar instructions to be the IEEE 754-2019 operations (thus, the !HONOR_SNANS checks in the back end). -- Joseph S. Myers jos...@codesourcery.com