diff options
author | Owen Anderson <resistor@mac.com> | 2012-08-15 18:28:45 +0000 |
---|---|---|
committer | Owen Anderson <resistor@mac.com> | 2012-08-15 18:28:45 +0000 |
commit | 352dfff4476b076e2c032d59ea0def3519a7a5c9 (patch) | |
tree | c986b285aee567d25a8fc9179bd050cf50586e04 /llvm/lib/Support/APFloat.cpp | |
parent | 73e325de558520985f3250adf8c413e96273561f (diff) | |
download | bcm5719-llvm-352dfff4476b076e2c032d59ea0def3519a7a5c9.tar.gz bcm5719-llvm-352dfff4476b076e2c032d59ea0def3519a7a5c9.zip |
Fix another roundToIntegral bug where very large values could become infinity. Problem and solution identified by Steve Canon.
llvm-svn: 161969
Diffstat (limited to 'llvm/lib/Support/APFloat.cpp')
-rw-r--r-- | llvm/lib/Support/APFloat.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/llvm/lib/Support/APFloat.cpp b/llvm/lib/Support/APFloat.cpp index b42a168bae7..ed261a4194c 100644 --- a/llvm/lib/Support/APFloat.cpp +++ b/llvm/lib/Support/APFloat.cpp @@ -1770,6 +1770,12 @@ APFloat::opStatus APFloat::roundToIntegral(roundingMode rounding_mode) { opStatus fs; assertArithmeticOK(*semantics); + // 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)) + return opOK; + // The algorithm here is quite simple: we add 2^(p-1), where p is the // precision of our format, and then subtract it back off again. The choice // of rounding modes for the addition/subtraction determines the rounding mode |