[9] RFR(S): 8144487: PhaseIdealLoop::build_and_optimize() must restore major_progress flag if skip_loop_opts is true

[Tobias Hartmann]

Thanks, Vladimir!

Best,
Tobias

On 18.12.2015 20:02, Vladimir Kozlov wrote:
> Fix is good.
> 
> Thanks,
> Vladimir
> 
> On 12/18/15 7:24 AM, Tobias Hartmann wrote:
>> Hi,
>>
>> please review the following patch.
>>
>> https://bugs.openjdk.java.net/browse/JDK-8144487
>> http://cr.openjdk.java.net/~thartmann/8144487/webrev.00/
>>
>> The fix for JDK-7107042 introduced a 'skip_loop_opts' flag for PhaseIdealLoop::build_and_optimize() to not execute loop optimizations before EA. We need to restore the major_progress flag before calling igvn.optimize() because other code depends on the fact that we don't execute more loop optimizations if major_progress() is not set (for example, in ConvI2LNode::Ideal).
>>
>> Thanks,
>> Tobias
>>