[Mlir-commits] [mlir] [mlir][IR] Require inherent symbol attributes for Symbol operations (PR #218920)

[Fabian Mora]

https://github.com/fabianmcg commented:

In general the direction LGTM, can we remove the default implementation in `getNameAttr` and the declaration and usage of `SymbolTable::getSymbolAttrName()`?
Since the name, visibility, all should be backed by the op, we should drop the lookup via attributes and rely on the interface methods to resolve to the proper storage in the op. 
Or is that out of scope and intended for a future patch?

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