RFR(T) : 8237953 : vmTestbase/jit/tiered/Test.java failure after JDK-8237798

[Igor Veresov]

Looks good.

igor



> On Jan 29, 2020, at 8:03 PM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> 
> as 8238049 got withdrawn, this is to be solved differently. http://cr.openjdk.java.net/~iignatyev//8237953/webrev.01 <http://cr.openjdk.java.net/~iignatyev//8237953/webrev.01> adds @requires to exclude the test from execution when JVMCI compiler is enabled.
> 
> -- Igor
> 
>> On Jan 29, 2020, at 11:23 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>> 
>> Good.
>> 
>> Vladimir K
>> 
>> On 1/29/20 11:20 AM, Igor Ignatyev wrote:
>>> http://cr.openjdk.java.net/~iignatyev//8237953/webrev.00
>>>> 2 lines changed: 2 ins; 0 del; 0 mod;
>>> Hi all,
>>> could you please review the small fix for vmTestbase/jit/tiered/Test? "TieredCompilation is disabled" case fails when the test is run w/ a JVMCI compiler b/c TieredCompilation is reenabled. to make test more robust, the patch explicitly sets CompilationMode to "high-only" in -TieredCompilation case and to "default" in +TieredCompilation. the patch depends on 8238049[1-3].
>>> webrev: http://cr.openjdk.java.net/~iignatyev//8237953/webrev.00
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8237953
>>> testing: vmTestbase/jit/tiered/Test on {linux, windows, macosx}-x64 w/ and w/o JVMCI compiler
>>> [1] https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2020-January/036850.html
>>> [2] https://bugs.openjdk.java.net/browse/JDK-8238049
>>> [3] http://cr.openjdk.java.net/~iignatyev/8238049/webrev.00/
>>> Thanks,
>>> -- Igor
>