[gold patch] Add --no-gnu-unique option to disable STB_GNU_UNIQUE

[Cary Coutant <ccoutant at google dot com>]

The use of STB_GNU_UNIQUE breaks a bunch of applications that expect
dlopen("lib", RTLD_LOCAL) to load "lib" in a local namespace. The
loader makes the subset of symbols that have STB_GNU_UNIQUE binding
global, regardless of the option to dlopen(), and the apps break
because of the unexpected sharing across namespaces. The attached
patch adds --[no-]gnu-unique options to control conversion of
STB_GNU_UNIQUE symbols into regular STB_GLOBAL symbols in the output
file.

OK?

-cary


	* options.h (class General_options): Add --[no-]gnu-unique options.
	* symtab.cc (Symbol_table::sized_write_globals): Convert
	STB_GNU_UNIQUE to STB_GLOBAL if --no-gnu-unique.

Attachment: gold-gnu-unique-patch.txt
Description: Text document