[Mlir-commits] [mlir] [mlir][dataflow] Use OpWithFlags skipRegions to replace opName when print op in SparseAnalysis.cpp (NFC) (PR #175418)

[lonely eagle]

https://github.com/linuxlonelyeagle created https://github.com/llvm/llvm-project/pull/175418

None

>From 97cbe95fe265db78b6a3ffbb5604f026cd25a871 Mon Sep 17 00:00:00 2001
From: linuxlonelyeagle <2020382038 at qq.com>
Date: Sun, 11 Jan 2026 08:59:06 +0000
Subject: [PATCH] use OpWithFlags(&op, OpPrintingFlags().skipRegions()) to
 replace op->getName.

---
 mlir/lib/Analysis/DataFlow/SparseAnalysis.cpp | 20 ++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/mlir/lib/Analysis/DataFlow/SparseAnalysis.cpp b/mlir/lib/Analysis/DataFlow/SparseAnalysis.cpp
index b9d861830dd38..9fa9329430736 100644
--- a/mlir/lib/Analysis/DataFlow/SparseAnalysis.cpp
+++ b/mlir/lib/Analysis/DataFlow/SparseAnalysis.cpp
@@ -67,12 +67,14 @@ AbstractSparseForwardDataFlowAnalysis::initialize(Operation *top) {
 
 LogicalResult
 AbstractSparseForwardDataFlowAnalysis::initializeRecursively(Operation *op) {
-  LDBG() << "Initializing recursively for operation: " << op->getName();
+  LDBG() << "Initializing recursively for operation: "
+         << OpWithFlags(op, OpPrintingFlags().skipRegions());
 
   // Initialize the analysis by visiting every owner of an SSA value (all
   // operations and blocks).
   if (failed(visitOperation(op))) {
-    LDBG() << "Failed to visit operation: " << op->getName();
+    LDBG() << "Failed to visit operation: "
+           << OpWithFlags(op, OpPrintingFlags().skipRegions());
     return failure();
   }
 
@@ -86,9 +88,11 @@ AbstractSparseForwardDataFlowAnalysis::initializeRecursively(Operation *op) {
           ->blockContentSubscribe(this);
       visitBlock(&block);
       for (Operation &op : block) {
-        LDBG() << "Recursively initializing nested operation: " << op.getName();
+        LDBG() << "Recursively initializing nested operation: "
+               << OpWithFlags(&op, OpPrintingFlags().skipRegions());
         if (failed(initializeRecursively(&op))) {
-          LDBG() << "Failed to initialize nested operation: " << op.getName();
+          LDBG() << "Failed to initialize nested operation: "
+                 << OpWithFlags(&op, OpPrintingFlags().skipRegions());
           return failure();
         }
       }
@@ -96,7 +100,7 @@ AbstractSparseForwardDataFlowAnalysis::initializeRecursively(Operation *op) {
   }
 
   LDBG() << "Successfully completed recursive initialization for operation: "
-         << op->getName();
+         << OpWithFlags(op, OpPrintingFlags().skipRegions());
   return success();
 }
 
@@ -428,7 +432,8 @@ static MutableArrayRef<OpOperand> operandsToOpOperands(OperandRange &operands) {
 
 LogicalResult
 AbstractSparseBackwardDataFlowAnalysis::visitOperation(Operation *op) {
-  LDBG() << "Visiting operation: " << op->getName() << " with "
+  LDBG() << "Visiting operation: "
+         << OpWithFlags(op, OpPrintingFlags().skipRegions()) << " with "
          << op->getNumOperands() << " operands and " << op->getNumResults()
          << " results";
 
@@ -563,7 +568,8 @@ AbstractSparseBackwardDataFlowAnalysis::visitOperation(Operation *op) {
     }
   }
 
-  LDBG() << "Using default visitOperationImpl for operation: " << op->getName();
+  LDBG() << "Using default visitOperationImpl for operation: "
+         << OpWithFlags(op, OpPrintingFlags().skipRegions());
   return visitOperationImpl(op, operandLattices, resultLattices);
 }