From f578f97b97c8b46ec85a81a186c55db9c5a3c8cf Mon Sep 17 00:00:00 2001 From: Joseph Myers Date: Mon, 12 Nov 2018 23:59:05 +0000 Subject: Fix armv7 build with GCC 9. Similar to the x86_64 build issues, glibc fails to build for armv7 with current mainline GCC because of aliases declared in the course of defining IFUNCs, which copy their attributes from a header declaration, ending up with fewer attributes than the (built-in) string function they alias: the relevant attributes (nonnull, leaf) are present on the header declaration, but elided therefrom when glibc itself if being built (whatever the reasons are for disabling the nonnull and leaf attributes in that case, and whether or not those reasons are actually still valid). This patch fixes the issue similarly to the x86_64 fix, by adding an addition __attribute_copy__ use (in this case, on the definition of arm_libc_ifunc_hidden_def). Tested with build-many-glibcs.py build for armeb-linux-gnueabi-be8. * sysdeps/arm/arm-ifunc.h [SHARED] (arm_libc_ifunc_hidden_def): Use __attribute_copy__ to copy attributes from name. --- sysdeps/arm/arm-ifunc.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'sysdeps/arm') diff --git a/sysdeps/arm/arm-ifunc.h b/sysdeps/arm/arm-ifunc.h index 80f4f0e3a3..71fb172b3b 100644 --- a/sysdeps/arm/arm-ifunc.h +++ b/sysdeps/arm/arm-ifunc.h @@ -27,7 +27,8 @@ #if defined SHARED # define arm_libc_ifunc_hidden_def(redirect_name, name) \ __hidden_ver1 (name, __GI_##name, redirect_name) \ - __attribute__ ((visibility ("hidden"))) + __attribute__ ((visibility ("hidden"))) \ + __attribute_copy__ (name) #else # define arm_libc_ifunc_hidden_def(redirect_name, name) #endif -- cgit 1.4.1