[PATCH] D24496: [ELF] - Implemented rpath-link command line option.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 22 03:44:06 PDT 2016


grimar added a comment.

In https://reviews.llvm.org/D24496#541673, @ruiu wrote:

> I was bitten by a subtle difference (or a bug) in the file search path code before, which is fixed in r273846. I can tell from that experience that it was not easy to find a bug. If the linker finds another file in a different directory in search paths, it tries to use that file instead of an intended one and eventually fail in a unresolved symbol error or something like that which seems totally unrelated to a search path problem. So my opinion is that we probably should implement what the GNU linker does to avoid such mess in the future. At least we need to understand the exact semantics of the GNU linker.


When prepared this patch, I just already forgot why we ignored it before, but now I remember: it was https://reviews.llvm.org/D18269,
we desided to ignore it like gold do that time.


https://reviews.llvm.org/D24496





More information about the llvm-commits mailing list