diff mbox series

Enable ifunc attribute by default for SPARC GNU/Linux

Message ID alpine.DEB.2.20.1709261801220.21154@digraph.polyomino.org.uk
State New
Headers show
Series Enable ifunc attribute by default for SPARC GNU/Linux | expand

Commit Message

Joseph Myers Sept. 26, 2017, 6:02 p.m. UTC 
Similar to other architectures with IFUNC binutils/glibc support, this
patch enables the ifunc attribute for SPARC GNU/Linux.  This is needed
for building glibc with the current checks on IFUNC resolver types
(and use of the attribute in glibc rather than manually created IFUNCs
is beneficial anyway because it results in better debug info).

Tested compilation of glibc (in conjunction with a glibc patch to
support using the attribute on SPARC) with build-many-glibcs.py.  I
have not run the GCC tests for SPARC.  OK to commit?

2017-09-26  Joseph Myers  <joseph@codesourcery.com>

	* config.gcc (default_gnu_indirect_function): Default to yes for
	sparc*-*-linux* with glibc.

Comments

Eric Botcazou Sept. 28, 2017, 1:52 p.m. UTC | #1 
> Similar to other architectures with IFUNC binutils/glibc support, this
> patch enables the ifunc attribute for SPARC GNU/Linux.  This is needed
> for building glibc with the current checks on IFUNC resolver types
> (and use of the attribute in glibc rather than manually created IFUNCs
> is beneficial anyway because it results in better debug info).
> 
> Tested compilation of glibc (in conjunction with a glibc patch to
> support using the attribute on SPARC) with build-many-glibcs.py.  I
> have not run the GCC tests for SPARC.  OK to commit?

I presume so, although I don't really understand all the consequences.
diff mbox series

Patch

Index: gcc/config.gcc
===================================================================
--- gcc/config.gcc	(revision 253204)
+++ gcc/config.gcc	(working copy)
@@ -3100,7 +3100,7 @@ 
         ;;
 *-*-linux*)
 	case ${target} in
-	aarch64*-* | i[34567]86-* | powerpc*-* | s390*-* | x86_64-*)
+	aarch64*-* | i[34567]86-* | powerpc*-* | s390*-* | sparc*-* | x86_64-*)
 		default_gnu_indirect_function=yes
 		;;
 	esac