[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