This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc 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 C11 conformance issues


On Wed, 25 Jul 2018, Florian Weimer wrote:

> On 07/25/2018 07:18 PM, Joseph Myers wrote:
> > On Wed, 25 Jul 2018, Florian Weimer wrote:
> > 
> > > On 07/25/2018 03:43 PM, Adhemerval Zanella wrote:
> > > >    linknamespace-libs-ISO11 = $(linknamespace-libs-isoc) \
> > > > -			   $(common-objpfx)nptl/libpthread.a
> > > > -linknamespace-libs-XPG4 = $(linknamespace-libs-isoc) \
> > > > -			  $(common-objpfx)crypt/libcrypt.a
> > > > +			   $(static-thread-library)
> > > 
> > > I've just realized this:
> > > 
> > > Linking with the static thread library obscures linknamespace issues in
> > > libc.a
> > > for symbols which are also defined in libpthread.a.  So this change is a
> > > bit
> > > questionable.
> > 
> > The linknamespace code looks at every definition of a strong undefined
> > symbol being resolved, not just the definition in a particular library.
> 
> So there is not a problem after all?

I don't think there's any problem with that part of the patch.

> What seems to be a problem is that libpthread.a must now be C11-clean for its
> C11 symbols (which is currently not the case on Hurd).  But fixing that is
> probably not a bad idea anyway.

Yes, if libpthread is used to provide some C11 symbols, it needs to be 
C11-clean for all the C11 symbols it provides (including those also 
present in libc).

-- 
Joseph S. Myers
joseph@codesourcery.com


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