Re: [Chicken-hackers] [PATCH] Fix performance bottleneck in compiling large files, add profiling option

[Peter Bex]

On Mon, Jan 23, 2012 at 11:19:14AM +0100, Felix wrote:
> > I considered that, but I think in some cases it would be useful to
> > be able to increase the limit, or remove it altogether.
> > Don't forget, it's not just the C comments but also the scrutinizer
> > messages that are affected by this option.
> > 
> > We could move the option to the "obscure options" section of the help :)
> > 
> 
> No, please no such option.

OK, no prob, then we'll just hardcode it.  I doubt people will find
it useful to increase the depth, anyway.

> > This could be done but I'm afraid that traversing all the way up the
> > tree also causes exponential cost.  A cutoff point ensures that there's
> > an upper bound for each item, and that upper bound can be seen as a
> > constant factor of the compilation as a whole.
> 
> Then cut it off, 20 should be enough.

How about the attached patch?

Cheers,
Peter
-- 
http://sjamaan.ath.cx
--
"The process of preparing programs for a digital computer
 is especially attractive, not only because it can be economically
 and scientifically rewarding, but also because it can be an aesthetic
 experience much like composing poetry or music."
                                                        -- Donald Knuth

0001-Limit-depth-for-procedure-nesting-reports-to-ensure-.patch
Description: Text document