commit c03642e9a86d9ed128676d5405a1906282d727ad
author Leonard Chan <leonardchan@google.com> Mon Aug 06 16:05:08 2018 +0000
committer Leonard Chan <leonardchan@google.com> Mon Aug 06 16:05:08 2018 +0000
tree c221a392e0bf810a25b78cc0f46c89c8525c0c48
parent 0d1b3934e27f72f706345678204a87f72007ed64

[Fixed Point Arithmetic] Fix for FixedPointValueToString

- Print negative numbers correctly
- Handle APInts of different sizes
- Add formal unit tests for FixedPointValueToString
- Add tests for checking correct printing when padding is set
- Restrict to printing in radix 10 since that's all we need for now

Differential Revision: https://reviews.llvm.org/D49945

llvm-svn: 339026

clang/lib/AST/Expr.cpp

diff --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp
index 7556c76..69adcbc 100644
--- a/clang/lib/AST/Expr.cpp
+++ b/clang/lib/AST/Expr.cpp
@@ -785,7 +785,7 @@
   // which is 43 characters.
   SmallString<64> S;
   FixedPointValueToString(
-      S, llvm::APSInt::getUnsigned(getValue().getZExtValue()), Scale, Radix);
+      S, llvm::APSInt::getUnsigned(getValue().getZExtValue()), Scale);
   return S.str();
 }