On Fri, Aug 17, 2012 at 6:17 AM, Richard Guenther <rguent...@suse.de> wrote:

> which means only 19516 reallocations and 350kb wasted memory.  The
> compile-time effects are neutral (name-lookup is 1.0+-0.05s on
> the reduced testcase) - a slightly different version which omitted
> the temporary vector but walked things twice was consistently slower
> (about 1.2s).

Funny; I found something similar 10 years ago :-/

> It seems we can end up with duplicates in the
> names chain of store_bindings (not sure if that is intended).
>

the chain is supposed to implement a stack, not a set.

-- Gaby

Reply via email to