[Mlir-commits] [mlir] [mlir] Use SymbolOpInterface to implement operateOnSymbol in test-symbol-uses pass (PR #172675)

[Mehdi Amini]

================
@@ -71,9 +71,8 @@ struct SymbolUsesPass
 
     // Walk nested symbols.
     SmallVector<func::FuncOp, 4> deadFunctions;
-    module.getBodyRegion().walk([&](Operation *nestedOp) {
-      if (isa<SymbolOpInterface>(nestedOp))
-        return operateOnSymbol(nestedOp, module, deadFunctions);
+    module.getBodyRegion().walk([&](SymbolOpInterface nestedOp) {
+      return operateOnSymbol(nestedOp, module, deadFunctions);
       return WalkResult::advance();
----------------
joker-eph wrote:

Isn't this dead?


```suggestion
```

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