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

[llvmlistbot at llvm.org]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: lonely eagle (linuxlonelyeagle)

<details>
<summary>Changes</summary>



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


1 Files Affected:

- (modified) mlir/lib/Analysis/DataFlow/SparseAnalysis.cpp (+13-7) 


``````````diff
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);
 }
 

``````````

</details>


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