[lld] [LLD] [MinGW] Ignore the -rpath option (PR #102886)

Martin Storsjö via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 13 22:37:58 PDT 2024


mstorsjo wrote:

> I wonder whether this is a significant build system issue worth working around on the linker side.
> 
> If newer build systems have been fixed, I certainly support having this for `release/19.x`, but I think we probably don't need it for `main`?

TBH I don't think it's a very significant and widespread thing - I've only got a handful of reports about this.

I don't see a big benefit in doing it as a one-off for only one release though, so either we don't do it at all, or we keep it. (The one user who pointed to older versions of Conan doing this was stuck on such an old version for other reasons, so nothing will change in 6 months I guess.) But I don't really see a big problem in keeping it around, especially if we add a warning to it. That keeps users informed that they should try to get rid of it, while smoothing out the corners for new users migrating to LLD.

So my suggestion would be to include it in main (either with or without a backport to 19.x - the main deal there is that we're quite late in the process already), but with a warning, and keep it there for the future too.

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


More information about the llvm-commits mailing list