[PATCH] D18269: [ELF] - -rpath-link "implemented"

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 19 03:16:31 PDT 2016


ruiu added a comment.

I agree with your argument that we should ignore that option. gold completely ignores that option, and the GNU ld's behavior to try to resolve undefined symbols in all shared object files at (static-)link time seems broken to me.

I don't think you need a test for this. Intended behavior of this patch is ignoring that option and it is trivially achieved by your change.


================
Comment at: ELF/Options.td:190
@@ -189,2 +189,3 @@
 def no_warn_mismatch : Flag<["--"], "no-warn-mismatch">;
+def rpath_link : Separate<["-"], "rpath-link">;
 def version_script : Separate<["--"], "version-script">;
----------------
I think both -rpath-link and --rpath-link are valid.


http://reviews.llvm.org/D18269





More information about the llvm-commits mailing list