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

[Albert]

Hi,

Vladimir, Rickard, thanks for looking at this. Rickard, I followed your 
suggestion
and made init_worklist iterative. Here is the new webrev:

http://cr.openjdk.java.net/~anoll/8039425/webrev.01/

Best,
Albert



On 04/10/2014 06:57 AM, Rickard Bäckman wrote:
> 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