This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] Lno branch merge part 3 -- ssa form updatingimprovements
On Mon, 2004-06-14 at 23:48, Zdenek Dvorak wrote:
> Yes; the method is not really much different from what
> rewrite_ssa_into_ssa does. The exceptions are that
> rewrite_ssa_into_ssa searches for new definitions by scanning all the
> statements (making it possibly a bit slower), and that it works for
> multiple definitions at a time (making it possibly a bit faster, and
> much easier to use then direct incremental algorithm).
>
Then let's keep them separate. They are conceptually different and we
need the abstraction of being able to insert new SSA definitions. So
far, I see no compelling reasons why we should make rewrite_into_ssa any
more complicated than it already is.
The patch is very intrusive in that respect and it will be difficult to
maintain. We will now have two SSA renaming algorithms, each slightly
different yet similar.
Am I missing anything? You are resisting the idea of having a separate
name insertion utility, but I don't really understand why.
Thanks. Diego.