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 | 7bb9992cceda7c0c0deca013661b131ee9686d8e (patch) | |
| tree | 7423ec99353591f420520203d6cb5f266fc06081 /llvm | |
| parent | 6e67bed48ccc70b73b8a041d2c651b15ec86cd35 (diff) | |
| download | bcm5719-llvm-7bb9992cceda7c0c0deca013661b131ee9686d8e.tar.gz bcm5719-llvm-7bb9992cceda7c0c0deca013661b131ee9686d8e.zip | |
Allow the asm printer to print fp128 values properly.
llvm-svn: 171866
Diffstat (limited to 'llvm')
| -rw-r--r-- | llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp | 21 | ||||
| -rw-r--r-- | llvm/test/CodeGen/ARM/fp128.ll | 10 | ||||
| -rw-r--r-- | llvm/test/CodeGen/PowerPC/fp128.ll | 8 |
3 files changed, 30 insertions, 9 deletions
diff --git a/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp b/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp index 9e43524ad01..aa64698644a 100644 --- a/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp +++ b/llvm/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/llvm/test/CodeGen/ARM/fp128.ll b/llvm/test/CodeGen/ARM/fp128.ll new file mode 100644 index 00000000000..ab476d43ac9 --- /dev/null +++ b/llvm/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/llvm/test/CodeGen/PowerPC/fp128.ll b/llvm/test/CodeGen/PowerPC/fp128.ll new file mode 100644 index 00000000000..9755f9ba6eb --- /dev/null +++ b/llvm/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 + |
