RFR(S): 8146096 - [TEST BUG] compiler/loopopts/UseCountedLoopSafepoints.java Timeouts

[Dmitrij Pochepko]

Hi,

Please take a look at v02.

I've rewritten this test. Now it launch vm with 
-XX:+UseCountedLoopSafepoints (restricting compilation to tested method 
with single simple counted loop) and checks that output of 
-XX:+PrintIdeal have edge SafePoint -> CountedLoopEnd

Then, launch the same with -XX:-UseCountedLoopSafepoints and checks that 
there is no such edge.

I've tested fix via rbt on all platforms.

webrev: http://cr.openjdk.java.net/~dpochepk/8146096/webrev.02/


Thanks,

Dmitrij

On 29.08.2016 19:28, Vladimir Kozlov wrote:
> I am not against marking test as stress but I think the test itself is 
> not good. It should be rewrote. I added comment to the JBS with 
> discussion during original 8146096 RFR.
>
> Thanks,
> Vladimir
>
> On 8/29/16 7:08 AM, Dmitrij Pochepko wrote:
>> Hi,
>>
>> please review small fix for 8146096 - [TEST BUG]
>> compiler/loopopts/UseCountedLoopSafepoints.java Timeouts
>>
>>
>> Test timeouts on slow platforms, so, this fix adds execution control
>> with respect to elapsed time. Also, test marked as stress.
>>
>> webrev: http://cr.openjdk.java.net/~dpochepk/8146096/webrev.01/
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8146096
>>
>>
>> I've tested this fix on linux-amd64.
>>
>>
>> Thanks,
>>
>> Dmitrij
>>