Fix unused errors in build without assertions (#872)

Xilinx · Jan 21, 2025 · d226668 · d226668
1 parent aec2938
commit d226668
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/mlir/lib/Conversion/AIRLoweringPass.cpp b/mlir/lib/Conversion/AIRLoweringPass.cpp
@@ -525,7 +525,7 @@ AIRChannelInterfaceToAIRRtConversionImpl(OpBuilder builder,
     strides.push_back(one_idx);
   }
   // Stride field implicit last element one
-  auto lastStrideConst = getConstantIntValue(strides.back());
+  [[maybe_unused]] auto lastStrideConst = getConstantIntValue(strides.back());
   assert(lastStrideConst && "the last stride is not static");
 
   strides.pop_back();

diff --git a/mlir/lib/Transform/AIRDependencyScheduleOpt.cpp b/mlir/lib/Transform/AIRDependencyScheduleOpt.cpp
@@ -1507,7 +1507,7 @@ struct UnrollScfParallel : public OpRewritePattern<scf::ParallelOp> {
     auto loc = rewriter.getUnknownLoc();
 
     for (auto lb : par.getLowerBound()) {
-      auto constLB = getConstantIntValue(lb);
+      [[maybe_unused]] auto constLB = getConstantIntValue(lb);
       assert(constLB && "non-static scf.parallel lb, NYI");
       assert(*constLB == 0 && "non-zero scf.parallel lb, NYI");
     }
@@ -3802,7 +3802,7 @@ class AIRFuseChannels
           return notMergeable;
       }
       // Merge by unpeeling into UB.
-      auto outerMostScfFor =
+      [[maybe_unused]] auto outerMostScfFor =
           dyn_cast<scf::ForOp>(a_loop_nest[outermostScfFor]->getParentOp());
       assert(outerMostScfFor);
       return mergeableToUB;

diff --git a/mlir/lib/Util/Util.cpp b/mlir/lib/Util/Util.cpp
@@ -1375,8 +1375,8 @@ air::writeAccessPattern(mlir::vector::TransferReadOp readOp) {
   OpBuilder builder(readOp);
   std::tuple<SmallVector<Value>, SmallVector<Value>, SmallVector<Value>>
       pattern;
-  auto vectorTy = llvm::cast<VectorType>(readOp.getVector().getType());
-  auto memrefTy = llvm::cast<MemRefType>(readOp.getSource().getType());
+  [[maybe_unused]] auto vectorTy = llvm::cast<VectorType>(readOp.getVector().getType());
+  [[maybe_unused]] auto memrefTy = llvm::cast<MemRefType>(readOp.getSource().getType());
   assert(vectorTy && "Not a vector");
   assert(memrefTy && "Not a memref");
   // Initialize wraps and strides based on the unshrunk memref shape.
@@ -1400,8 +1400,8 @@ air::writeAccessPattern(mlir::vector::TransferWriteOp writeOp) {
   OpBuilder builder(writeOp);
   std::tuple<SmallVector<Value>, SmallVector<Value>, SmallVector<Value>>
       pattern;
-  auto memrefTy = llvm::cast<MemRefType>(writeOp.getSource().getType());
-  auto vectorTy = llvm::cast<VectorType>(writeOp.getVector().getType());
+  [[maybe_unused]] auto memrefTy = llvm::cast<MemRefType>(writeOp.getSource().getType());
+  [[maybe_unused]] auto vectorTy = llvm::cast<VectorType>(writeOp.getVector().getType());
   assert(memrefTy && "Not a memref");
   assert(vectorTy && "Not a vector");
   // Initialize wraps and strides based on the unshrunk memref shape.