[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