On 02/24/2018 09:21 AM, Laurent Vivier wrote:
> I think the change should be in fact:
> 
> @@ -4549,10 +4555,7 @@ int64_t floatx80_to_int64(floatx80 a,
> float_status *status)
>      if ( shiftCount <= 0 ) {
>          if ( shiftCount ) {
>              float_raise(float_flag_invalid, status);
> -            if (    ! aSign
> -                 || (    ( aExp == 0x7FFF )
> -                      && ( aSig != LIT64( 0x8000000000000000 ) ) )
> -               ) {
> +            if (!aSign || floatx80_is_any_nan(a)) {
>                  return LIT64( 0x7FFFFFFFFFFFFFFF );
>              }
>              return (int64_t) LIT64( 0x8000000000000000 );
> 
> Do you agree?

Yep, looks good.


r~

Reply via email to