On 2026-01-28 12:34, Richard Henderson wrote:
> 
> I think the "internal" functions should already assume any_nan.
> 
> Otherwise you're double-testing it in the quiet_nan tests.
> 
> Perhaps the intent would be clearer by naming them <type>_nan_is_snan.
> 
> Perhaps clearer to merge the expressions:
> 
>     return float16_is_any_nan(a) && !float16_nan_is_snan(a)
> and
>     return float16_is_any_nan(a) && float16_nan_is_snan(a)
> 
> for the is_quiet_nan and is_signaling_nan functions respectively?
> 
> 
> r~

Hi Richard,

I agree with your suggestions and I will update v3 for this part.

Thanks,
rnax

Reply via email to