From ace9103ebefcece3e26c69735dd0087bb973579c Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Sun, 7 Apr 2002 08:37:11 +0000 Subject: [PATCH] Fix bug: test/Regression/Other/2002-04-07-HexFloatConstants.ll git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@2141 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/VMCore/Constants.cpp | 22 ++++++++++++++++++++-- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git a/lib/VMCore/Constants.cpp b/lib/VMCore/Constants.cpp index d9a2fa74314..410276683d1 100644 --- a/lib/VMCore/Constants.cpp +++ b/lib/VMCore/Constants.cpp @@ -13,7 +13,6 @@ #include "llvm/Analysis/SlotCalculator.h" #include "Support/StringExtras.h" #include -#include using std::map; using std::pair; @@ -159,8 +158,27 @@ std::string ConstantUInt::getStrValue() const { return utostr(Val.Unsigned); } +// ConstantFP::getStrValue - We would like to output the FP constant value in +// exponential notation, but we cannot do this if doing so will lose precision. +// Check here to make sure that we only output it in exponential format if we +// can parse the value back and get the same value. +// std::string ConstantFP::getStrValue() const { - return ftostr(Val); + std::string StrVal = ftostr(Val); + double TestVal = atof(StrVal.c_str()); // Reparse stringized version! + if (TestVal == Val) + return StrVal; + + // Otherwise we could not reparse it to exactly the same value, so we must + // output the string in hexadecimal format! + // + // Behave nicely in the face of C TBAA rules... see: + // http://www.nullstone.com/htmls/category/aliastyp.htm + // + char *Ptr = (char*)&Val; + assert(sizeof(double) == sizeof(uint64_t) && sizeof(double) == 8 && + "assuming that double is 64 bits!"); + return "0x"+utohexstr(*(uint64_t*)Ptr); } std::string ConstantArray::getStrValue() const { -- 2.34.1