release/19.x: [NFC] fix build failure (#100993)

[llvmbot]

Backport 40b4fd7

Requested by: @mgorny

[llvmbot]

@arsenm What do you think about merging this PR to the release branch?

[llvmbot]

@llvm/pr-subscribers-llvm-analysis

Author: None (llvmbot)

Changes

Backport 40b4fd7

Requested by: @mgorny

---

Full diff: https://github.com/llvm/llvm-project/pull/112551.diff

1 Files Affected:

• (modified) llvm/lib/Analysis/ConstantFolding.cpp (+2-2)

diff --git a/llvm/lib/Analysis/ConstantFolding.cpp b/llvm/lib/Analysis/ConstantFolding.cpp
index df75745645e049..ff30fece5fce93 100644
--- a/llvm/lib/Analysis/ConstantFolding.cpp
+++ b/llvm/lib/Analysis/ConstantFolding.cpp
@@ -1784,8 +1784,8 @@ Constant *ConstantFoldFP(double (*NativeFP)(double), const APFloat &V,
 }
 
 #if defined(HAS_IEE754_FLOAT128) && defined(HAS_LOGF128)
-Constant *ConstantFoldFP128(long double (*NativeFP)(long double),
-                            const APFloat &V, Type *Ty) {
+Constant *ConstantFoldFP128(float128 (*NativeFP)(float128), const APFloat &V,
+                            Type *Ty) {
   llvm_fenv_clearexcept();
   float128 Result = NativeFP(V.convertToQuad());
   if (llvm_fenv_testexcept()) {

[arsenm]

I'm assuming this isn't an ABI break in any cases where this builds?

[mgorny]

CC @chenzheng1030 @aaronpuchert

[aaronpuchert]

I'm assuming this isn't an ABI break in any cases where this builds?

The function is not declared in any header file, in fact it sits in an (unfortunately relatively large) anonymous namespace (starting here at line 1732). So it should be fine.

[mgorny]

@arsenm, does the above comment answer your concerns? If so, could you please approve it before the next release.

[github-actions]

@mgorny (or anyone else). If you would like to add a note about this fix in the release notes (completely optional). Please reply to this comment with a one or two sentence description of the fix. When you are done, please add the release:note label to this PR.