[9] RFR(S): 8039425: Code cleanup: PhaseIterGVN::optimize()

[Rickard Bäckman]

Albert,

I think it looks good. (Not a Reviewer...).
One thing I would have liked to see is making init_worklist
iterative instead of recursive. I'm fine with it the way it is though.

Thanks
/R

On 04/09, Albert wrote:
> Hi,
> 
> could I get reviews for this patch?
> 
> *Bug:*
> https://bugs.openjdk.java.net/browse/JDK-8039425
> 
> *Problem:*
> The code in PhaseIterGVN::optimize() is hard to read.
> 
> *Solution:*
> This code cleanup aims at improving code readability.
> 
> *Webrev:*
> http://cr.openjdk.java.net/~anoll/8039425/webrev.00/
> 
> Many thanks in advance,
> Albert
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140410/6031a6d9/signature.asc>