diff --git a/clang/include/clang/AST/Expr.h b/clang/include/clang/AST/Expr.h index 381345586462..9a76823ff263 100644 --- a/clang/include/clang/AST/Expr.h +++ b/clang/include/clang/AST/Expr.h @@ -271,17 +271,11 @@ public: bool isExact() const { return IsExact; } - /// getValueAsDouble - This returns the value as an inaccurate double. Note - /// that this may cause loss of precision, but is useful for debugging dumps - /// etc. - double getValueAsDouble() const { - // FIXME: We need something for long double here. - if (cast(getType())->getKind() == BuiltinType::Float) - return Value.convertToFloat(); - else - return Value.convertToDouble(); - } - + /// getValueAsApproximateDouble - This returns the value as an inaccurate + /// double. Note that this may cause loss of precision, but is useful for + /// debugging dumps, etc. + double getValueAsApproximateDouble() const; + virtual SourceRange getSourceRange() const { return SourceRange(Loc); } static bool classof(const Stmt *T) { diff --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp index a89fbd621a56..78c305051ee4 100644 --- a/clang/lib/AST/Expr.cpp +++ b/clang/lib/AST/Expr.cpp @@ -22,6 +22,16 @@ using namespace clang; // Primary Expressions. //===----------------------------------------------------------------------===// +/// getValueAsApproximateDouble - This returns the value as an inaccurate +/// double. Note that this may cause loss of precision, but is useful for +/// debugging dumps, etc. +double FloatingLiteral::getValueAsApproximateDouble() const { + llvm::APFloat V = getValue(); + V.convert(llvm::APFloat::IEEEdouble, llvm::APFloat::rmNearestTiesToEven); + return V.convertToDouble(); +} + + StringLiteral::StringLiteral(const char *strData, unsigned byteLength, bool Wide, QualType t, SourceLocation firstLoc, SourceLocation lastLoc) : diff --git a/clang/lib/AST/StmtDumper.cpp b/clang/lib/AST/StmtDumper.cpp index 082d2a225957..ae6ca40460af 100644 --- a/clang/lib/AST/StmtDumper.cpp +++ b/clang/lib/AST/StmtDumper.cpp @@ -335,7 +335,7 @@ void StmtDumper::VisitIntegerLiteral(IntegerLiteral *Node) { } void StmtDumper::VisitFloatingLiteral(FloatingLiteral *Node) { DumpExpr(Node); - fprintf(F, " %f", Node->getValueAsDouble()); + fprintf(F, " %f", Node->getValueAsApproximateDouble()); } void StmtDumper::VisitStringLiteral(StringLiteral *Str) { diff --git a/clang/lib/AST/StmtPrinter.cpp b/clang/lib/AST/StmtPrinter.cpp index a740facf76fc..5e347435b633 100644 --- a/clang/lib/AST/StmtPrinter.cpp +++ b/clang/lib/AST/StmtPrinter.cpp @@ -577,7 +577,7 @@ void StmtPrinter::VisitIntegerLiteral(IntegerLiteral *Node) { } void StmtPrinter::VisitFloatingLiteral(FloatingLiteral *Node) { // FIXME: print value more precisely. - OS << Node->getValueAsDouble(); + OS << Node->getValueAsApproximateDouble(); } void StmtPrinter::VisitImaginaryLiteral(ImaginaryLiteral *Node) {