Patch to speedup haifa scheduler for big basic blocks

Nix nix@esperi.demon.co.uk
Sun Oct 8 11:06:00 GMT 2000


Vladimir Makarov <vmakarov@cygnus.com> writes:

> >         free (true_dependency_cache);
> >         true_dependency_cache = NULL;
> >+       free (anti_dependency_cache);
> >+       anti_dependency_cache = NULL;
> >+       free (output_dependency_cache);
> >+       output_dependency_cache = NULL;
> >+ #ifdef ENABLE_CHECKING
> >+       free (output_dependency_cache);
>                 ^^^^^^^^^^^^^^^^^^^^^^^^
> Shouldn't that be forward_dependency_cache?
> 
> Thanks, Franz.  Sure it should be forward_dependency_cache.  It is
> strange that bootsrap test worked fine on i386 and sparc.

Not that strange; free(NULL) is guaranteed to work, and
output_dependency_cache is NULL there. So you just had a memory leak.

-- 
`Naturally, a sysadmin's entire person is holy.  We have the power to kill
 daemons.' --- Mike Sphar


More information about the Gcc-patches mailing list