This is the mail archive of the 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: [v3] Filter out basever symbols on Solaris

Hi Paolo,

>>>> Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> writes:
>>>>> diff --git a/libstdc++-v3/scripts/ b/libstdc++-v3/scripts/
>>>>> --- a/libstdc++-v3/scripts/
>>>>> +++ b/libstdc++-v3/scripts/
>>>>> @@ -49,9 +49,12 @@ SunOS)
>>>>>     if readelf --help | grep -- --wide > /dev/null; then
>>>>>       readelf="$readelf --wide"
>>>>>     fi
>>>>> +  # Omit _DYNAMIC etc. for consistency with, only
>>>>> +  # present on Solaris.
>>>>>     ${readelf} ${lib} |\
>>>>>     sed -e 's/ \[<other>: [A-Fa-f0-9]*\] //' -e '/\.dynsym/,/^$/p;d' |\
>>>>>     egrep -v ' (LOCAL|UND) ' |\
>>>>> +  egrep -v '(_DYNAMIC|_GLOBAL_OFFSET_TABLE_|_PROCEDURE_LINKAGE_TABLE_|_edata|_end|_etext)$' |\
>>>> Shouldn't the regexp also be anchored (with a space) at the beginning?
>>> This works indeed.  I was a bit worried if a space was guaranteed at
>>> that position, rather than general whitespace.  Consider the patch
>>> amended accordingly.
>> Any word on this patch?  Is has remained unreviewed for a week.
> Sorry about the delay. If you can convince me that the change is guaranteed
> to affect only Solaris, then you will not need a specific approval.

Unless those symbols were explicitly exported in config/abi/pre/gnu*.ver
(which they are not, otherwise the x86_64-unknown-linux-bootstrap would
have shown a failure), they couldn't make it into  They
only occur on Solaris because gld explicitly adds them itself.


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]