maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[APInt] Remove an unneeded extra temporary APInt from toString. 

Turns out udivrem can write its output to the same location as one of its inputs so the extra temporary isn't needed.

llvm-svn: 302772
This commit is contained in:
Craig Topper 2017-05-11 07:10:43 +00:00
parent f3fde7e851
commit c59ced36aa
1 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/lib/Support/APInt.cpp
View File

@ -1953,15 +1953,11 @@ void APInt::toString(SmallVectorImpl<char> &Str, unsigned Radix,
} else {
APInt divisor(Tmp.getBitWidth(), Radix);
APInt APdigit;
APInt tmp2(Tmp.getBitWidth(), 0);
while (Tmp.getBoolValue()) {
udivrem(Tmp, divisor, tmp2, APdigit);
udivrem(Tmp, divisor, Tmp, APdigit);
unsigned Digit = (unsigned)APdigit.getZExtValue();
assert(Digit < Radix && "divide failed");
Str.push_back(Digits[Digit]);
// Move the quotient into Tmp and move the old allocation of Tmp into
// tmp2 to be used on the next loop iteration.
std::swap(Tmp, tmp2);
}
}