Skip to content

Conversation

@cathyzhyi
Copy link
Contributor

No description provided.

@llvmbot
Copy link
Member

llvmbot commented Dec 18, 2025

@llvm/pr-subscribers-mlir-execution-engine

@llvm/pr-subscribers-mlir

Author: Yi Zhang (cathyzhyi)

Changes

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

1 Files Affected:

  • (modified) mlir/lib/ExecutionEngine/APFloatWrappers.cpp (+1)
diff --git a/mlir/lib/ExecutionEngine/APFloatWrappers.cpp b/mlir/lib/ExecutionEngine/APFloatWrappers.cpp
index 9deb900fbe35d..b30a9072c0094 100644
--- a/mlir/lib/ExecutionEngine/APFloatWrappers.cpp
+++ b/mlir/lib/ExecutionEngine/APFloatWrappers.cpp
@@ -212,6 +212,7 @@ _mlir_apfloat_fused_multiply_add(int32_t semantics, uint64_t operand,
       multiplicand_, addend_, llvm::RoundingMode::NearestTiesToEven);
   assert(stat == llvm::APFloatBase::opOK &&
          "expected fusedMultiplyAdd status to be OK");
+  (void)stat;
   return operand_.bitcastToAPInt().getZExtValue();
 }
 

Copy link
Contributor

@makslevental makslevental left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@cathyzhyi cathyzhyi merged commit 188bf27 into llvm:main Dec 18, 2025
13 checks passed
mahesh-attarde pushed a commit to mahesh-attarde/llvm-project that referenced this pull request Dec 19, 2025
valadaptive pushed a commit to valadaptive/llvm-project that referenced this pull request Dec 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants