Index: llvm/lib/Support/APFloat.cpp =================================================================== --- llvm/lib/Support/APFloat.cpp +++ llvm/lib/Support/APFloat.cpp @@ -2242,6 +2242,16 @@ if (!X86SpecialNan && semantics == &semX87DoubleExtended) APInt::tcSetBit(significandParts(), semantics->precision - 1); + // If we are truncating NaN, it is possible that we shifted out all of the + // set bits in the NaN payload. But NaN must remain NaN, so some bit in the + // significand must be set (otherwise it is Inf). Set the quiet bit (the MSB + // of the significand). + if (APInt::tcIsZero(significandParts(), newPartCount)) { + assert(shift < 0 && "Should not lose NaN payload on extend"); + makeQuiet(); + *losesInfo = true; + } + // gcc forces the Quiet bit on, which means (float)(double)(float_sNan) // does not give you back the same bits. This is dubious, and we // don't currently do it. You're really supposed to get Index: llvm/test/Transforms/InstSimplify/ConstProp/cast.ll =================================================================== --- llvm/test/Transforms/InstSimplify/ConstProp/cast.ll +++ llvm/test/Transforms/InstSimplify/ConstProp/cast.ll @@ -39,19 +39,22 @@ ret float %i } -; https://llvm.org/PR43907 +; https://llvm.org/PR43907 - make sure that a NaN doesn't morph into Inf. define float @nan_f64_trunc() { ; CHECK-LABEL: @nan_f64_trunc( -; CHECK-NEXT: ret float 0x7FF0000000000000 +; CHECK-NEXT: ret float 0x7FF8000000000000 ; %f = fptrunc double 0x7FF0000000000001 to float ret float %f } +; Verify again with a vector type. NaN value may depend on whether any +; of the original payload bits are still set. + define <2 x half> @nan_v2f32_trunc() { ; CHECK-LABEL: @nan_v2f32_trunc( -; CHECK-NEXT: ret <2 x half> +; CHECK-NEXT: ret <2 x half> ; %f = fptrunc <2 x float> to <2 x half> ret <2 x half> %f