[Mlir-commits] [mlir] [mlir] Use getAttr to replace getInherentAttr in the test-symbol-uses pass (PR #172675)

[Mehdi Amini]

================
@@ -59,7 +59,7 @@ struct SymbolUsesPass
               symbolUse.getUser()->getParentOp(), symbolUse.getSymbolRef())) {
         symbolUse.getUser()->emitRemark()
             << "found use of symbol : " << symbolUse.getSymbolRef() << " : "
-            << *symbol->getInherentAttr(SymbolTable::getSymbolAttrName());
+            << symbol->getAttr(SymbolTable::getSymbolAttrName());
----------------
joker-eph wrote:

Seems like the "launch_func" is the culprit, it does not seem to be defined as `SymbolUserOpInterface`, so I'm not even sure why it is visited at all. We seem to have some inconsistency here.

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