[Mlir-commits] [mlir] [MLIR] Enable import of non self referential alias scopes (PR #121987)

[Tobias Gysi]

================
@@ -473,9 +495,18 @@ ModuleImport::processAliasScopeMetadata(const llvm::MDNode *node) {
       StringAttr description = nullptr;
       if (!aliasScope.getName().empty())
         description = builder.getStringAttr(aliasScope.getName());
-      auto aliasScopeOp = builder.getAttr<AliasScopeAttr>(
-          DistinctAttr::create(builder.getUnitAttr()),
-          cast<AliasScopeDomainAttr>(it->second), description);
+      AliasScopeAttr aliasScopeOp;
+      if (verifySelfRef(scope))
----------------
gysit wrote:

I would probably also create the id attribute before and then create the scope attribute as above.

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