Re: C++ PATCH [3.3] Improve name-lookup time

[Matt Austern <austern at apple dot com>]

On Saturday, May 10, 2003, at 02:26 AM, Gabriel Dos Reis wrote:

> gcc@integrable-solutions.net writes:
>
> [...]
>
> | This patch was successfully bootstrapped and regtested on an
> | i686-pc-linux-gnu with no regression.
> |
> | OK for 3.3 and mainline?
>
> That patch has been reviewed, commented on by Mark Mitchell and 
> approved
> for 3.3.1 and mainline.  This version integrates comments made by
> Mark.  It also integrates a correction for an initialization bug found
> by Matt Austern.  I'm not going to apply it right now because I have
> compile-time gain issues to resolve with Matt.  I'm posting it there so
> that people can experiment with it and report numbers.

The "experiment with it and report numbers" part is important.  One of 
the
issues here is that Gaby and I don't seem to be seeing the same numbers
for the improvement due to this patch.  He and I have done a fair amount
of back and forth offline, and our tests are similar enough that we 
shouldn't
be seeing such large differences are we are.

If other people could do timing tests with and without Gaby's patch, it 
might
help to nail down what's going on.

			--Matt