This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix up --enable-initfini-array autodetection in configure (PR bootstrap/50237)


Paolo Bonzini <bonzini@gnu.org> writes:

>>  changequote(,)dnl
>> +if test "x${build}" = "x${target}" && test "x${build}" = "x${host}"; then
>> +  case "${target}" in
>> +    *-*-solaris2*)
>> +      # ld.so.1 and ld are guaranteed to be updated in lockstep, so check
>> +      # ld version number for use in ld.so.1 feature checks even if a
>> +      # different linker is configured.
>> +      #
>> +      # See below for the version number formats.
>> +      sun_ld_ver=`/usr/ccs/bin/ld -V 2>&1`
>> +      if echo "$sun_ld_ver" | grep 'Solaris Link Editors' > /dev/null; then
>> +	sun_ld_vers=`echo $sun_ld_ver | sed -n \
>> +	  -e 's,^.*: 5\.[0-9][0-9]*-\([0-9]\.[0-9][0-9]*\).*$,\1,p'`
>> +	sun_ld_vers_major=`expr "$sun_ld_vers" : '\([0-9]*\)'`
>> +	sun_ld_vers_minor=`expr "$sun_ld_vers" : '[0-9]*\.\([0-9]*\)'`
>> +      fi
>> +      ;;
>> +  esac
>> +fi
>
> This must be a separate macro in acinclude.m4 that is AC_REQUIREd from
> gcc_AC_INITFINI_ARRAY.

Ok, will do.

> Can you test Linux too?  The change of -e0 looks good, but I'd rather check
> on the actual system.

I can test on x86_64-unknown-linux-gnu, just not on my laptop at home.
Actually, Jakub suggested the change in the PR since Sun ld complains
about -e 0.

	Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]