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

Tobias Gysi llvmlistbot at llvm.org
Tue Jan 7 13:09:28 PST 2025


================
@@ -438,8 +445,14 @@ ModuleImport::processAliasScopeMetadata(const llvm::MDNode *node) {
     if (aliasDomain->getNumOperands() >= 2)
       if (auto *operand = dyn_cast<llvm::MDString>(aliasDomain->getOperand(1)))
         description = builder.getStringAttr(operand->getString());
-    return builder.getAttr<AliasScopeDomainAttr>(
-        DistinctAttr::create(builder.getUnitAttr()), description);
+    Attribute idattr;
+    if (verifySelfRef(aliasDomain))
+      idattr = DistinctAttr::create(builder.getUnitAttr());
+    else {
+      auto Name = cast<llvm::MDString>(aliasDomain->getOperand(0));
+      idattr = builder.getStringAttr(Name->getString());
----------------
gysit wrote:

```suggestion
      auto name = cast<llvm::MDString>(aliasDomain->getOperand(0));
      idattr = builder.getStringAttr(name->getString());
```
nit: 

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


More information about the Mlir-commits mailing list