[Mlir-commits] [mlir] [mlir][arith] Fix variable only used by assert (PR #188461)

[Yi Zhang]

https://github.com/cathyzhyi updated https://github.com/llvm/llvm-project/pull/188461

>From 2d0087f34ce7f7b64a9be4a0e2350c0902cbf93d Mon Sep 17 00:00:00 2001
From: Yi Zhang <cathyzhyi at google.com>
Date: Wed, 25 Mar 2026 07:10:44 -0400
Subject: [PATCH] [mlir][arith] Fix variable only used by assert

---
 mlir/lib/Conversion/ArithToLLVM/ArithToLLVM.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/mlir/lib/Conversion/ArithToLLVM/ArithToLLVM.cpp b/mlir/lib/Conversion/ArithToLLVM/ArithToLLVM.cpp
index a0346ec6f4fb6..64ed672878f7b 100644
--- a/mlir/lib/Conversion/ArithToLLVM/ArithToLLVM.cpp
+++ b/mlir/lib/Conversion/ArithToLLVM/ArithToLLVM.cpp
@@ -279,8 +279,8 @@ struct ConvertFOpLowering : public ConvertOpToLLVMPattern<arith::ConvertFOp> {
 
     // Only bf16 <-> f16 conversions are supported. There is currently no other
     // pair of FP types that are valid LLVM types.
-    auto srcType = getElementTypeOrSelf(op.getIn().getType());
-    auto dstType = getElementTypeOrSelf(op.getType());
+    [[maybe_unused]] auto srcType = getElementTypeOrSelf(op.getIn().getType());
+    [[maybe_unused]] auto dstType = getElementTypeOrSelf(op.getType());
     assert((srcType.isBF16() && dstType.isF16()) ||
            (srcType.isF16() && dstType.isBF16()) &&
                "only bf16 <-> f16 conversions are supported");