Re: CVS commit: src/sys/lib/libsa

[christos%zoulas.com@localhost (Christos Zoulas)]

On Mar 27,  7:36am, david%l8s.co.uk@localhost (David Laight) wrote:
-- Subject: Re: CVS commit: src/sys/lib/libsa

| On Wed, Mar 26, 2008 at 09:42:42PM +0000, Christos Zoulas wrote:
| > >
| > >Stuff for boot/bootxx often uses STANDALONE
| > 
| > If size is important it should also be compiled with -Os, and then
| > __OPTIMIZE_SIZE__ is defined. If you don't care about size, you are
| > not using -Os, and you get speed instead :-) It seems that some part
| > of the boot code does not use -Os, so I assume it does not care about
| > size.
| 
| I don't think you can use __OPTIMIZE_SIZE__ in .S files, and the libc
| functions (in libkern) always use the .S versions in preference to the
| .c ones. The 'optimised' .S versions are larger than the .c ones
| (although it is trivially possible to write much smaller .S ones).

Yes, but libsa links the C ones now like it did before, grabbing them
directly from common/lib/libc/string. What I said still holds that not
all boot code is compiled with -Os as it is supposed to.

christos