cvs commit: src/sys/kern init_main.c kern_malloc.c md5c.c subr_autoconf.c subr_mbuf.c subr_prf.c tty_subr.c vfs_cluster.c vfs_subr.c

[Bosko Milekic]

On Wed, Jul 23, 2003 at 12:56:34AM +0200, Poul-Henning Kamp wrote:
...
> I agree that there are exceptional cases, I agree that we need some
> sort of __inline_damnit, but I still request that we only used it when
> we know for a fact that there is an actual benefit.
> 
> And the only two criteria I think are trivial to use for proving an
> actual benefit is:
> 	1. less code is generated.
> 	2. it runs faster in tests.
> 
> I am very reluctant to accept "a speculated benefit" when there is
> as strong a counter indication as 2k extra code segment.

  What about the example I (and Bruce) posted in the other thread?

  That example does not necessarily meet your criteria but it certainly
  is valid as it minimizes code duplication and does not necessarily
  impact badly on performance (think of a case where the tradeoff
  between having it as a function call or inlined in a couple of places
  is evened out). 

> -- 
> Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
> phk at FreeBSD.ORG         | TCP/IP since RFC 956
> FreeBSD committer       | BSD since 4.3-tahoe    
> Never attribute to malice what can adequately be explained by incompetence.

-- 
Bosko Milekic  *  bmilekic at technokratis.com  *  bmilekic at FreeBSD.org
TECHNOkRATIS Consulting Services  *  http://www.technokratis.com/