Fix -Wundef warnings for SHARED

The definition of SHARED is tested with #ifdef pretty much everywhere
apart from these few places. The tlsdesc.c code seems to be copy and
pasted to a few architectures and there is one instance in the hppa
startup code.

ChangeLog:

2014-07-09  Will Newton  <will.newton@linaro.org>

	* sysdeps/aarch64/tlsdesc.c (_dl_unmap): Test SHARED with #ifdef.
	* sysdeps/arm/tlsdesc.c (_dl_unmap): Likewise.
	* sysdeps/i386/tlsdesc.c (_dl_unmap): Likewise.
	* sysdeps/x86_64/tlsdesc.c (_dl_unmap): Likewise.
	* sysdeps/hppa/start.S (_start): Likewise.
This commit is contained in:
Will Newton 2014-07-07 15:14:26 +01:00
parent 6ee3eab429
commit 82374e65d7
6 changed files with 13 additions and 5 deletions

View File

@ -1,3 +1,11 @@
2014-07-09 Will Newton <will.newton@linaro.org>
* sysdeps/aarch64/tlsdesc.c (_dl_unmap): Test SHARED with #ifdef.
* sysdeps/arm/tlsdesc.c (_dl_unmap): Likewise.
* sysdeps/i386/tlsdesc.c (_dl_unmap): Likewise.
* sysdeps/x86_64/tlsdesc.c (_dl_unmap): Likewise.
* sysdeps/hppa/start.S (_start): Likewise.
2014-07-09 Siddhesh Poyarekar <siddhesh@redhat.com> 2014-07-09 Siddhesh Poyarekar <siddhesh@redhat.com>
* sunrpc/xdr.c (xdr_string): Initialize SIZE to 0. * sunrpc/xdr.c (xdr_string): Initialize SIZE to 0.

View File

@ -147,7 +147,7 @@ _dl_unmap (struct link_map *map)
{ {
_dl_unmap_segments (map); _dl_unmap_segments (map);
#if SHARED #ifdef SHARED
if (map->l_mach.tlsdesc_table) if (map->l_mach.tlsdesc_table)
htab_delete (map->l_mach.tlsdesc_table); htab_delete (map->l_mach.tlsdesc_table);
#endif #endif

View File

@ -149,7 +149,7 @@ _dl_unmap (struct link_map *map)
{ {
_dl_unmap_segments (map); _dl_unmap_segments (map);
#if SHARED #ifdef SHARED
/* _dl_unmap is only called for dlopen()ed libraries, for which /* _dl_unmap is only called for dlopen()ed libraries, for which
calling free() is safe, or before we've completed the initial calling free() is safe, or before we've completed the initial
relocation, in which case calling free() is probably pointless, relocation, in which case calling free() is probably pointless,

View File

@ -93,7 +93,7 @@ _start:
stw %r23, -56(%sp) stw %r23, -56(%sp)
/* Need to setup 1, 4, 5, and 7th arguments */ /* Need to setup 1, 4, 5, and 7th arguments */
#if SHARED #ifdef SHARED
/* load main (1st argument) */ /* load main (1st argument) */
addil LT'.Lpmain, %r19 addil LT'.Lpmain, %r19
ldw RT'.Lpmain(%r1), %r26 ldw RT'.Lpmain(%r1), %r26

View File

@ -261,7 +261,7 @@ _dl_unmap (struct link_map *map)
{ {
_dl_unmap_segments (map); _dl_unmap_segments (map);
#if SHARED #ifdef SHARED
if (map->l_mach.tlsdesc_table) if (map->l_mach.tlsdesc_table)
htab_delete (map->l_mach.tlsdesc_table); htab_delete (map->l_mach.tlsdesc_table);
#endif #endif

View File

@ -139,7 +139,7 @@ _dl_unmap (struct link_map *map)
{ {
_dl_unmap_segments (map); _dl_unmap_segments (map);
#if SHARED #ifdef SHARED
/* _dl_unmap is only called for dlopen()ed libraries, for which /* _dl_unmap is only called for dlopen()ed libraries, for which
calling free() is safe, or before we've completed the initial calling free() is safe, or before we've completed the initial
relocation, in which case calling free() is probably pointless, relocation, in which case calling free() is probably pointless,