Subject: Re: inlining & size
To: Simon Burge <simonb@wasabisystems.com>
From: David Brownlee <abs@NetBSD.ORG>
List: tech-kern
Date: 09/20/2000 09:45:12
On Tue, 19 Sep 2000, Simon Burge wrote:

> > 	Is there any reason why we shouldn't add the above makeoptions to
> > 	all port's INSTALL configs?
> 
> Because -mmemcpy is MIPS specific :-)
> 
	Thats a pretty compelling reason :)

> But all INSTALL configs should probably use -Os.  Quick summary, with:
> 
> 	thoreau:src/sys/arch 8> grep -l -e -Os */conf/INSTAL*
> 	i386/conf/INSTALL
> 	i386/conf/INSTALL_SMALL
> 	i386/conf/INSTALL_TINY
> 	macppc/conf/INSTALL
> 	news68k/conf/INSTALL
> 	pc532/conf/INSTALL
> 	pmax/conf/INSTALL
> 	x68k/conf/INSTALL
> 	x68k/conf/INSTALL_SMALL
> 
> without:
> 
> 	thoreau:src/sys/arch 9> grep -L -e -Os */conf/INSTAL*
> 	alpha/conf/INSTALL
> 	amiga/conf/INSTALL
> 	bebox/conf/INSTALL
> 	mac68k/conf/INSTALL
> 	newsmips/conf/INSTALL
> 	sparc/conf/INSTALL
> 	sparc/conf/INSTALL-SYSINST
> 	sun3/conf/INSTALL
> 	sun3/conf/INSTALL3X
> 	vax/conf/INSTALL

	Modulus the point raised by masanobu@iij.ad.jp, that -Os implies
	-O2, so we should only add -O2 on platforms that are known to be
	bugfree for -O2, which would be
		alpha, amiga, mac68k, newsmips, sparc, sun3 and vax
		(excluding bebox).
	Though given that macppc includes -Os, maybe we should just do it
	across the board :)

	Unless anyone objects, I'll add -Os to all INSTALL files, except
	bebox, where I'll add it commented out...

                David/absolute
			       -- www.netbsd.org: A pmap for every occasion --