RFR(XXXS): 8166761: Compiler testing in tier2 should be optimized to finish in 20 minutes.

[Vladimir Kozlov]

On 11/16/16 8:48 AM, Vladimir Kozlov wrote:
> You should add hotspot_not_fast_compiler_closed for closed tests which
> are not part of hotspot_fast_compiler_closed

I take my comment out. Changes I good.

I was confused by hotspot_not_fast_compiler definition. I thought 
:hotspot_compiler here only includes open tests as it is defined in open 
test/TEST.groups. But it also includes all closed/compiler tests when 
they are present.
So hotspot_not_fast_compiler_closed is not needed.

Thanks,
Vladimir

>
> Thanks,
> Vladimir
>
> On 11/16/16 4:27 AM, Leonid Mesnik wrote:
>> Hi.
>>
>> Could you please review following tiny fix which introduce new group
>> of compiler tests. It allows to run execute all
>> tests 'hotspot_compiler' as several jobs and get results faster.
>>
>> Neither product or test code is affected by this fix.
>>
>> diff -r f5c44b9b4ff9 test/TEST.groups
>> --- a/test/TEST.groups  Mon Oct 17 05:44:43 2016 -0700
>> +++ b/test/TEST.groups  Wed Nov 16 15:14:58 2016 +0300
>> @@ -330,6 +330,13 @@
>>  hotspot_fast_compiler_closed = \
>>    sanity/ExecuteInternalVMTests.java
>>
>> +hotspot_not_fast_compiler = \
>> +  :hotspot_compiler \
>> +  -:hotspot_fast_compiler_1 \
>> +  -:hotspot_fast_compiler_2 \
>> +  -:hotspot_fast_compiler_3 \
>> +  -:hotspot_fast_compiler_closed
>> +
>>  hotspot_fast_gc_1 = \
>>    gc/g1/
>>
>>
>> Leonid
>>