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

[Rui Ueyama via llvm-commits]

ruiu added a comment.

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.


================
Comment at: ELF/Options.td:150
@@ +149,3 @@
+
+def rpath_link_eq: J<"rpath-link=">, HelpText<"Add DIR to link time shared library search path">;
+
----------------
Define `rpath_link_eq` as an alias for `rpath_link`.

  def alias_rpath_link_eq: J<"rpath-link=">, Alias<rpath_link>;


https://reviews.llvm.org/D24496