This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: PATCH to libstdc++ to use __cxa_thread_atexit_impl if available
- From: "Carlos O'Donell" <codonell at redhat dot com>
- To: Siddhesh Poyarekar <spoyarek at redhat dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Jason Merrill <jason at redhat dot com>, gcc-patches at gcc dot gnu dot org, libstdc++ <libstdc++ at gcc dot gnu dot org>, Siddhesh Poyarekar <siddhesh at redhat dot com>
- Date: Tue, 22 Jan 2013 23:09:13 -0500
- Subject: Re: PATCH to libstdc++ to use __cxa_thread_atexit_impl if available
- References: <tf05569bjnaym80cv49cp49j.1358900658199@email.android.com>
On 01/22/2013 07:24 PM, Siddhesh Poyarekar wrote:
> Cc'ing Carlos on this so that he's aware of it.
>
> Siddhesh
>
> Jakub Jelinek <jakub@redhat.com> wrote:
>
> On Sat, Jan 19, 2013 at 12:22:23PM -0500, Jason Merrill wrote:
>> Siddhesh has a patch to implement the thread atexit functionality in
>> glibc in order to integrate better with the dynamic loader and run
>> the cleanups in the correct order. Once it's available there, this
>> patch will make the copy in libsupc++ use it. The main
>> __cxa_thread_atexit function will always live in libsupc++, however,
>> in order to maintain ABI compatibility between releases of
>> libstdc++.
>>
>> Does this configure change look right, or should it go in linkage.m4
>> somewhere?
>>
>> I think I'll hold off checking this in until Siddhesh's patch goes
>> into glibc.
>
> Yeah, it should make it into glibc first (no idea why is it taking so long),
> otherwise we can't be sure it won't be implemented differently in glibc.
>
> Jakub
>
If there is a patch that needs to be reviewed immediately
then Siddhesh should ping that patch and CC me so I can
do a timely review. I'll look into this patch tomorrow.
In general we lack qualified reviewers in glibc.
We try our best to get through the incoming patches,
but given the new friendliness of the community
we're flooded with patches, but not with reviewers.
Please have patience has the community grows up.
Cheers,
Carlos.