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

Vladimir Kozlov Vladimir.Kozlov at Sun.COM
Wed Aug 12 13:52:57 PDT 2009

Tom Rodriguez wrote:
>>>> (and corresponding nreg_lo mapping for register pair).
>>>> May be it needs additional comment in your changes.
>>> It's got a comment to that effect already.  What kind of comment were 
>>> you thinking?
>> Something like this:
>> // Clear out a dead value mapped by the register to update
>> // the register->node mapping even if 'n' defines the same value.
> Where do you want this comment?

You can just update the new comment you added by replacing
the first sentence ?:

// Clear out a dead value before starting.


More information about the hotspot-compiler-dev mailing list