review (S) for 6862863: C2 compiler fails in elide_copy()

[Vladimir Kozlov]

Tom Rodriguez wrote:
> http://cr.openjdk.java.net/~never/6862863

Few questions:

Why yank_if_dead() checks only in(1) if it is also dead
and not other inputs which could be dead also after old
node is removed? should we fix it?

Also from the current code it is not clear that at the end
it does next for this nreg:

               regnd.map(nreg,n);
               value.map(nreg,val);

(and corresponding nreg_lo mapping for register pair).
May be it needs additional comment in your changes.

Also in the new method you use "regnd->at(nreg)"
when in other places regnd and value are passed as
references and "regnd[nreg]". Can you do the same?


Thanks,
Vladimir