[PATCH] Remove useless loop from alias.c

Daniel Berlin dberlin@dberlin.org
Mon Nov 27 14:07:00 GMT 2006


On 11/25/06, Steven Bosscher <stevenb.gcc@gmail.com> wrote:
> Hi,
>
> This patch removes an expensive loop in alias.c:init_alias_analysis.
> The comment before the loop already says it:
>
> -     This loop may not be needed any longer now that the main loop does
> -     a better job at propagating alias information.  */
>
> Well, it's not needed as far as I can tell.  I've counted the number
> of passes in this loop for ~600 preprocessed C and C++ files from GCC
> itself, and there was *never* more than one pass, i.e. nothing changed
> in the loop So even if this is still a useful loop, it's questionable
> whether the cost/benefit ratio justifies keeping the loop.

This is good enough numbers for me :)

The patch is okay, thanks!



More information about the Gcc-patches mailing list