diff options
| author | Tim Northover <Tim.Northover@arm.com> | 2013-01-08 16:56:23 +0000 |
|---|---|---|
| committer | Tim Northover <Tim.Northover@arm.com> | 2013-01-08 16:56:23 +0000 |
| commit | 90f011e0baa68ea02b80e8b913f7120537ce05aa (patch) | |
| tree | 553389fe32c30dbecc4036105800f8ce3ddeaada | |
| parent | 3c00336c7ee13f78d1ecc23fafa3571e5c76110c (diff) | |
Allow the asm printer to print fp128 values properly.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@171866 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/CodeGen/AsmPrinter/AsmPrinter.cpp | 21 | ||||
| -rw-r--r-- | test/CodeGen/ARM/fp128.ll | 10 | ||||
| -rw-r--r-- | test/CodeGen/PowerPC/fp128.ll | 8 |
3 files changed, 30 insertions, 9 deletions
diff --git a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp index 9e43524ad0..aa64698644 100644 --- a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp +++ b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp @@ -1784,27 +1784,30 @@ static void emitGlobalConstantFP(const ConstantFP *CFP, unsigned AddrSpace, return; } - if (CFP->getType()->isX86_FP80Ty()) { + if (CFP->getType()->isX86_FP80Ty() || CFP->getType()->isFP128Ty()) { // all long double variants are printed as hex // API needed to prevent premature destruction APInt API = CFP->getValueAPF().bitcastToAPInt(); const uint64_t *p = API.getRawData(); if (AP.isVerbose()) { // Convert to double so we can print the approximate val as a comment. - APFloat DoubleVal = CFP->getValueAPF(); - bool ignored; - DoubleVal.convert(APFloat::IEEEdouble, APFloat::rmNearestTiesToEven, - &ignored); - AP.OutStreamer.GetCommentOS() << "x86_fp80 ~= " - << DoubleVal.convertToDouble() << '\n'; + SmallString<8> StrVal; + CFP->getValueAPF().toString(StrVal); + + const char *TyNote = CFP->getType()->isFP128Ty() ? "fp128 " : "x86_fp80 "; + AP.OutStreamer.GetCommentOS() << TyNote << StrVal << '\n'; } + // The 80-bit type is made of a 64-bit and 16-bit value, the 128-bit has 2 + // 64-bit words. + uint32_t TrailingSize = CFP->getType()->isFP128Ty() ? 8 : 2; + if (AP.TM.getDataLayout()->isBigEndian()) { - AP.OutStreamer.EmitIntValue(p[1], 2, AddrSpace); + AP.OutStreamer.EmitIntValue(p[1], TrailingSize, AddrSpace); AP.OutStreamer.EmitIntValue(p[0], 8, AddrSpace); } else { AP.OutStreamer.EmitIntValue(p[0], 8, AddrSpace); - AP.OutStreamer.EmitIntValue(p[1], 2, AddrSpace); + AP.OutStreamer.EmitIntValue(p[1], TrailingSize, AddrSpace); } // Emit the tail padding for the long double. diff --git a/test/CodeGen/ARM/fp128.ll b/test/CodeGen/ARM/fp128.ll new file mode 100644 index 0000000000..ab476d43ac --- /dev/null +++ b/test/CodeGen/ARM/fp128.ll @@ -0,0 +1,10 @@ +; RUN: llc -march=arm < %s | FileCheck --check-prefix=LITTLEENDIAN %s + +@var = global fp128 0xL00000000000000008000000000000000 + +; CHECK-LITTLEENDIAN: var: +; CHECK-LITTLEENDIAN-NEXT: .long 0 @ fp128 -0 +; CHECK-LITTLEENDIAN-NEXT: .long 0 +; CHECK-LITTLEENDIAN-NEXT: .long 0 +; CHECK-LITTLEENDIAN-NEXT: .long 2147483648 + diff --git a/test/CodeGen/PowerPC/fp128.ll b/test/CodeGen/PowerPC/fp128.ll new file mode 100644 index 0000000000..9755f9ba6e --- /dev/null +++ b/test/CodeGen/PowerPC/fp128.ll @@ -0,0 +1,8 @@ +; RUN: llc -march=ppc64 < %s | FileCheck --check-prefix=BIGENDIAN %s + +@var = global fp128 0xL00000000000000008000000000000000 + +; CHECK-BIGENDIAN: var: +; CHECK-BIGENDIAN-NEXT: .quad -9223372036854775808 # fp128 -0 +; CHECK-BIGENDIAN-NEXT: .quad 0 + |