[Mlir-commits] [mlir] [mlir][llvm] Add `LLVM_DependentLibrariesAttr` (PR #133385)
Tobias Gysi
llvmlistbot at llvm.org
Fri Mar 28 09:35:20 PDT 2025
================
@@ -563,6 +563,24 @@ LogicalResult ModuleImport::convertLinkerOptionsMetadata() {
return success();
}
+LogicalResult ModuleImport::convertDependentLibrariesMetadata() {
+ for (const llvm::NamedMDNode &named : llvmModule->named_metadata()) {
+ if (named.getName() != "llvm.dependent-libraries")
+ continue;
+ SmallVector<StringRef> libraries;
+ for (const llvm::MDNode *node : named.operands()) {
+ if (node->getNumOperands() == 1)
+ if (auto *mdString =
+ llvm::dyn_cast<llvm::MDString>(node->getOperand(0)))
----------------
gysit wrote:
```suggestion
dyn_cast<llvm::MDString>(node->getOperand(0)))
```
ultra nit: the namespace should not be necessary
https://github.com/llvm/llvm-project/pull/133385
More information about the Mlir-commits
mailing list