[compiler-rt] [builtins] Support building the 128-bit float functions on ld80 platforms (PR #68132)

[Sean Perry via llvm-commits]

perry-ca wrote:

Yeah.  The code will be same, so I’d just use #elif __LDBL_MANT_DIG__ == 113 || (__FLT_RADIX__ == 16 && __LDBL_MANT_DIG__ == 28).

Are you good with the other changes?

From: Alexander Richardson ***@***.***>
Sent: Thursday, January 4, 2024 2:36 PM
To: llvm/llvm-project ***@***.***>
Cc: Sean Perry ***@***.***>; Mention ***@***.***>
Subject: [EXTERNAL] Re: [llvm/llvm-project] [builtins] Support building the 128-bit float functions on ld80 platforms (PR #68132)

It sounds to me like the easiest way forward here would be to add another #elif __FLT_RADIX__ == 16 && __LDBL_MANT_DIG__ == 28 just before #elif __LDBL_MANT_DIG__ == 113 that defines the require typedefs and macros? — Reply to
ZjQcmQRYFpfptBannerStart
This Message Is From an External Sender
This message came from outside your organization.
    Report Suspicious  <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/PjiDSg!2U-qSTa06VIRMu4YcIkOZYJv87-YdAlWKJZhcNecT-hiiTgHmoKubkyWfa0jfbevUIuGUR7IYUdtspLTWJCll_uRjRWZQvrDjt4C$>   ‌
ZjQcmQRYFpfptBannerEnd

It sounds to me like the easiest way forward here would be to add another #elif __FLT_RADIX__ == 16 && __LDBL_MANT_DIG__ == 28 just before #elif __LDBL_MANT_DIG__ == 113 that defines the require typedefs and macros?

—
Reply to this email directly, view it on GitHub<https://github.com/llvm/llvm-project/pull/68132#issuecomment-1877655200>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AJQT7WGLWWXI66DGGCN3F5LYM4AAFAVCNFSM6AAAAAA5RKX7RCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZXGY2TKMRQGA>.
You are receiving this because you were mentioned.Message ID: ***@***.******@***.***>>


https://github.com/llvm/llvm-project/pull/68132