[clang] [driver] return immediately in `addArchSpecificRPath` and `getArchSpecificLibPaths` on AIX (PR #134520)
David Tenty via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 28 11:17:22 PDT 2025
================
@@ -1252,6 +1252,9 @@ void tools::addArchSpecificRPath(const ToolChain &TC, const ArgList &Args,
options::OPT_fno_rtlib_add_rpath, false))
return;
+ if (TC.getTriple().isOSAIX()) // AIX doesn't support -rpath option.
----------------
daltenty wrote:
nit: We did add `-rpath` support to the clang driver, essentially we fold all the `rpath` options into a `blibpath` for the library.
That said, I'm not convinced we really want to add anything here anyway right now, so maybe a `TODO` would be appropriate.
```suggestion
if (TC.getTriple().isOSAIX()) // TODO: AIX doesn't support -rpath option.
```
https://github.com/llvm/llvm-project/pull/134520
More information about the cfe-commits
mailing list