summaryrefslogtreecommitdiff
path: root/lib/Support/APInt.cpp
diff options
context:
space:
mode:
authorEli Friedman <eli.friedman@gmail.com>2011-10-07 23:40:49 +0000
committerEli Friedman <eli.friedman@gmail.com>2011-10-07 23:40:49 +0000
commit9eb6b4d91b83448ec818089754c74bbdcf7dfd7a (patch)
treebc23b23e593554a3b01ef19bded7a7f71ff22db6 /lib/Support/APInt.cpp
parentd2fdb4a28560688be5cbc5b2b0c305ec7207cbd0 (diff)
downloadllvm-9eb6b4d91b83448ec818089754c74bbdcf7dfd7a.tar.gz
llvm-9eb6b4d91b83448ec818089754c74bbdcf7dfd7a.tar.bz2
llvm-9eb6b4d91b83448ec818089754c74bbdcf7dfd7a.tar.xz
Fix APInt::operator*= so that it computes the correct result for large integers where there is unsigned overflow. Fix APFloat::toString so that it doesn't depend on the incorrect behavior in common cases (and computes the correct result in some rare cases). Fixes PR11086.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@141441 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Support/APInt.cpp')
-rw-r--r--lib/Support/APInt.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/Support/APInt.cpp b/lib/Support/APInt.cpp
index 6eadaafd5f..3774c5223c 100644
--- a/lib/Support/APInt.cpp
+++ b/lib/Support/APInt.cpp
@@ -386,6 +386,7 @@ APInt& APInt::operator*=(const APInt& RHS) {
clearAllBits();
unsigned wordsToCopy = destWords >= getNumWords() ? getNumWords() : destWords;
memcpy(pVal, dest, wordsToCopy * APINT_WORD_SIZE);
+ clearUnusedBits();
// delete dest array and return
delete[] dest;
@@ -471,7 +472,7 @@ APInt APInt::operator*(const APInt& RHS) const {
return APInt(BitWidth, VAL * RHS.VAL);
APInt Result(*this);
Result *= RHS;
- return Result.clearUnusedBits();
+ return Result;
}
APInt APInt::operator+(const APInt& RHS) const {