[Mlir-commits] [mlir] [mlir] Use llvm::find_if (NFC) (PR #139912)

Kazu Hirata llvmlistbot at llvm.org
Wed May 14 07:41:21 PDT 2025


https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/139912

None

>From 19e0f955dc26a8db77621e4491905d9c944530c0 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Mon, 12 May 2025 09:20:33 -0700
Subject: [PATCH] [mlir] Use llvm::find_if (NFC)

---
 .../Affine/Transforms/AffineDataCopyGeneration.cpp       | 9 ++++-----
 mlir/lib/Dialect/SparseTensor/IR/SparseTensorDialect.cpp | 4 ++--
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/mlir/lib/Dialect/Affine/Transforms/AffineDataCopyGeneration.cpp b/mlir/lib/Dialect/Affine/Transforms/AffineDataCopyGeneration.cpp
index 62c1857e4b1da..c11f1bca5d49d 100644
--- a/mlir/lib/Dialect/Affine/Transforms/AffineDataCopyGeneration.cpp
+++ b/mlir/lib/Dialect/Affine/Transforms/AffineDataCopyGeneration.cpp
@@ -126,11 +126,10 @@ void AffineDataCopyGeneration::runOnBlock(Block *block,
   // moment; we do a check later and report an error with location info.
 
   // Get to the first load, store, or for op (that is not a copy nest itself).
-  auto curBegin =
-      std::find_if(block->begin(), block->end(), [&](Operation &op) {
-        return isa<AffineLoadOp, AffineStoreOp, AffineForOp>(op) &&
-               copyNests.count(&op) == 0;
-      });
+  auto curBegin = llvm::find_if(*block, [&](Operation &op) {
+    return isa<AffineLoadOp, AffineStoreOp, AffineForOp>(op) &&
+           copyNests.count(&op) == 0;
+  });
 
   // Create [begin, end) ranges.
   auto it = curBegin;
diff --git a/mlir/lib/Dialect/SparseTensor/IR/SparseTensorDialect.cpp b/mlir/lib/Dialect/SparseTensor/IR/SparseTensorDialect.cpp
index fcbef0c14739f..2196199816292 100644
--- a/mlir/lib/Dialect/SparseTensor/IR/SparseTensorDialect.cpp
+++ b/mlir/lib/Dialect/SparseTensor/IR/SparseTensorDialect.cpp
@@ -791,7 +791,7 @@ LogicalResult SparseTensorEncodingAttr::verify(
     return emitError() << "unexpected coordinate bitwidth: " << crdWidth;
 
   // Verify every COO segment.
-  auto *it = std::find_if(lvlTypes.begin(), lvlTypes.end(), isSingletonLT);
+  auto *it = llvm::find_if(lvlTypes, isSingletonLT);
   while (it != lvlTypes.end()) {
     if (it == lvlTypes.begin() ||
         !(it - 1)->isa<LevelFormat::Compressed, LevelFormat::LooseCompressed>())
@@ -829,7 +829,7 @@ LogicalResult SparseTensorEncodingAttr::verify(
   }
 
   // TODO: audit formats that actually are supported by backend.
-  if (auto it = std::find_if(lvlTypes.begin(), lvlTypes.end(), isNOutOfMLT);
+  if (auto it = llvm::find_if(lvlTypes, isNOutOfMLT);
       it != std::end(lvlTypes)) {
     if (it != lvlTypes.end() - 1)
       return emitError() << "expected n_out_of_m to be the last level type";



More information about the Mlir-commits mailing list