[lld] [lld] load rpaths from tbd files (PR #134925)

Vy Nguyen via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 9 10:33:51 PDT 2025


================
@@ -1879,6 +1879,9 @@ DylibFile::DylibFile(const InterfaceFile &interface, DylibFile *umbrella,
   installName = saver().save(interface.getInstallName());
   compatibilityVersion = interface.getCompatibilityVersion().rawValue();
   currentVersion = interface.getCurrentVersion().rawValue();
+  for (auto rpath : interface.rpaths())
----------------
oontvoo wrote:

```suggestion
  for (const auto& rpath : interface.rpaths())
```

nit: This could  possibly trigger "performance-for-range clangtidy", so maybe should just avoid the unnecessary copy?

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


More information about the llvm-commits mailing list