On Wed, Feb 22, 2023 at 04:23:33AM -0800, Yash Shinde wrote:
> From: Khem Raj <raj.k...@gmail.com>
> 
> Adapter from
> 
> https://gcc.gnu.org/ml/gcc-patches/2015-05/msg00899.html
> 
> This fix was debated but hasnt been applied gcc upstream since
> they expect musl to support '@' in symbol versioning which is
> a sun/gnu versioning extention. This patch however avoids the
> need for the '@' symbols at all

This is an undesirable ABI change:
-__cpu_indicator_init@GCC_4.8.0 FUNC GLOBAL DEFAULT
-__cpu_model@GCC_4.8.0 OBJECT GLOBAL DEFAULT 16
+__cpu_indicator_init@@GCC_4.8.0 FUNC GLOBAL DEFAULT
+__cpu_model@@GCC_4.8.0 OBJECT GLOBAL DEFAULT 16

So, this is certainly unacceptable.

> libgcc/Changelog:
> 
> 2015-05-11  Szabolcs Nagy  <szabolcs.n...@arm.com>
> 
>         * config/i386/cpuinfo.c (__cpu_indicator_init_local): Add.
>         (__cpu_indicator_init@GCC_4.8.0, __cpu_model@GCC_4.8.0): Remove.
> 
>         * config/i386/t-linux (HOST_LIBGCC2_CFLAGS): Remove -DUSE_ELF_SYMVER.
> 
> gcc/Changelog:
> 
> 2015-05-11  Szabolcs Nagy  <szabolcs.n...@arm.com>
> 
>         * config/i386/i386-expand.c (ix86_expand_builtin): Make 
> __builtin_cpu_init
>         call __cpu_indicator_init_local instead of __cpu_indicator_init.

        Jakub

Reply via email to