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

[Albert]

Hi,

can I get a second review for this patch?

Best,
Albert

On 04/10/2014 06:28 PM, Vladimir Kozlov wrote:
> Okay.
>
> Vladimir
>
> On 4/10/14 5:55 AM, Albert wrote:
>> 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
>>