From: Andrey Bokhanko Date: Thu, 5 Nov 2015 12:43:09 +0000 (+0000) Subject: [x86] Additional small fix for MCU psABI support X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7805dc89fa42e76d586fe44993f8327f71927790;p=clang [x86] Additional small fix for MCU psABI support This patch fixes one more thing in MCU psABI support: LongDoubleWidth should be set to 64. Differential Revision: http://reviews.llvm.org/D14285 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@252156 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Basic/Targets.cpp b/lib/Basic/Targets.cpp index 70fa6b3f1b..a88c614e1f 100644 --- a/lib/Basic/Targets.cpp +++ b/lib/Basic/Targets.cpp @@ -2335,8 +2335,7 @@ class X86TargetInfo : public TargetInfo { public: X86TargetInfo(const llvm::Triple &Triple) : TargetInfo(Triple) { BigEndian = false; - LongDoubleFormat = Triple.isOSIAMCU() ? &llvm::APFloat::IEEEdouble - : &llvm::APFloat::x87DoubleExtended; + LongDoubleFormat = &llvm::APFloat::x87DoubleExtended; } unsigned getFloatEvalMethod() const override { // X87 evaluates with 80 bits "long double" precision. @@ -3624,6 +3623,11 @@ public: IntPtrType = SignedInt; RegParmMax = 3; + if (getTriple().isOSIAMCU()) { + LongDoubleWidth = 64; + LongDoubleFormat = &llvm::APFloat::IEEEdouble; + } + // Use fpret for all types. RealTypeUsesObjCFPRet = ((1 << TargetInfo::Float) | (1 << TargetInfo::Double) | diff --git a/test/CodeGen/x86_32-arguments-iamcu.c b/test/CodeGen/x86_32-arguments-iamcu.c index bac2fb71a0..12fafdf4f5 100644 --- a/test/CodeGen/x86_32-arguments-iamcu.c +++ b/test/CodeGen/x86_32-arguments-iamcu.c @@ -57,6 +57,6 @@ st12_t retLargeStruct(int i1, st12_t r) { return r; } // CHECK-LABEL: define i32 @varArgs(i32 inreg %i1, ...) int varArgs(int i1, ...) { return i1; } -// CHECK-LABEL: define double @longDoubleArg(double %ld1) +// CHECK-LABEL: define double @longDoubleArg(double inreg %ld1) long double longDoubleArg(long double ld1) { return ld1; }