[patch] remove .size directives from libc/sysdeps/linux/powerpc/crtn.S

Peter Mazinger ps.m at gmx.net
Tue May 3 21:35:30 UTC 2011


-------- Original-Nachricht --------
> Datum: Tue, 3 May 2011 21:33:45 +0200
> Von: Bernhard Reutner-Fischer <rep.dot.nop at gmail.com>
> An: Douglas Mencken <dougmencken at gmail.com>
> CC: uclibc at uclibc.org, Peter Mazinger <ps.m at gmx.net>
> Betreff: Re: [patch] remove .size directives from libc/sysdeps/linux/powerpc/crtn.S

> On Mar 23, 2011 9:27 PM, "Douglas Mencken" <dougmencken at gmail.com> wrote:
> >
> > > crti.o has these already, why would we need references in crtn.o as
> well? When we link some app/lib, crti.o and crtn.o are used together, so
> the
> functions are around, I think we need only the .section init/fini.
> >
> > This requires some testing. I may perform that tomorrow.
> >
> > > Does the ppc assembler fail on all *.S files that use .size or only if
> the function is not also defined in the same file? So .size would be
> correct
> (assembly-wise in crti.S, but not crtn.S), I know that we should not need
> it
> at all (glibc compiles these *initfini* with inhibit-size-directive)
> 
> X86_64/crtn.c should do
> .size _init,.-.init
> if anything (I.e. use the size of the section, not a non-existing function
> name). Since that would conflict with the previous size in crti, I guess
> that we should remove it from crtn altogether -- manually setting the size
> of the section is of no use, isn't it.

I am for removing all .size directive in crt*.S files (one of my later patches for the future branch would do that, but do it in master if you
want to

Peter
> 
> >
> > Currently, 'as' fails on any .size directive. crti.S doesn't have any
> > .size at all.
> > _______________________________________________
> > uClibc mailing list
> > uClibc at uclibc.org
> > http://lists.busybox.net/mailman/listinfo/uclibc

-- 
NEU: FreePhone - kostenlos mobil telefonieren und surfen!			
Jetzt informieren: http://www.gmx.net/de/go/freephone


More information about the uClibc mailing list