mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-08-21 18:29:46 +00:00
APFloat::roundToIntegral: Special values don't keep the exponent value up to date, don't rely on it.
Add a couple of unit tests for special floats. Fixes 13929, found by MemorySanitizer. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@164698 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
50b7d70707
commit
3e7735fe1c
@ -1775,7 +1775,7 @@ APFloat::opStatus APFloat::roundToIntegral(roundingMode rounding_mode) {
|
||||
// If the exponent is large enough, we know that this value is already
|
||||
// integral, and the arithmetic below would potentially cause it to saturate
|
||||
// to +/-Inf. Bail out early instead.
|
||||
if (exponent+1 >= (int)semanticsPrecision(*semantics))
|
||||
if (category == fcNormal && exponent+1 >= (int)semanticsPrecision(*semantics))
|
||||
return opOK;
|
||||
|
||||
// The algorithm here is quite simple: we add 2^(p-1), where p is the
|
||||
|
@ -689,6 +689,23 @@ TEST(APFloatTest, roundToIntegral) {
|
||||
P = R;
|
||||
P.roundToIntegral(APFloat::rmNearestTiesToEven);
|
||||
EXPECT_EQ(R.convertToDouble(), P.convertToDouble());
|
||||
|
||||
P = APFloat::getZero(APFloat::IEEEdouble);
|
||||
P.roundToIntegral(APFloat::rmTowardZero);
|
||||
EXPECT_EQ(0.0, P.convertToDouble());
|
||||
P = APFloat::getZero(APFloat::IEEEdouble, true);
|
||||
P.roundToIntegral(APFloat::rmTowardZero);
|
||||
EXPECT_EQ(-0.0, P.convertToDouble());
|
||||
P = APFloat::getNaN(APFloat::IEEEdouble);
|
||||
P.roundToIntegral(APFloat::rmTowardZero);
|
||||
EXPECT_TRUE(IsNAN(P.convertToDouble()));
|
||||
P = APFloat::getInf(APFloat::IEEEdouble);
|
||||
P.roundToIntegral(APFloat::rmTowardZero);
|
||||
EXPECT_TRUE(IsInf(P.convertToDouble()) && P.convertToDouble() > 0.0);
|
||||
P = APFloat::getInf(APFloat::IEEEdouble, true);
|
||||
P.roundToIntegral(APFloat::rmTowardZero);
|
||||
EXPECT_TRUE(IsInf(P.convertToDouble()) && P.convertToDouble() < 0.0);
|
||||
|
||||
}
|
||||
|
||||
TEST(APFloatTest, getLargest) {
|
||||
|
Loading…
Reference in New Issue
Block a user