RFR: JDK-8238586: [TESTBUG] vmTestbase/jit/tiered/Test.java failed when TieredCompilation is disabled

[Jia Huang]

Hi Igor,

Thanks for your review.

Could you please sponsor it?

Thanks a lot.
Best regards,
Jia.

在 2020/2/7 23:47, Igor Ignatev 写道:
> I’d consider it as trivial.
>
> — Igor
>
>> On Feb 6, 2020, at 11:22 PM, Jia Huang <huangjia at loongson.cn> wrote:
>>
>> Thanks Igor.
>>
>> Is it a trivial patch and OK to be pushed?
>> Or shall we wait for one more review?
>>
>> Thanks a lot.
>> Best regards,
>> Jia.
>>
>>> 在 2020/2/7 14:06, Igor Ignatyev 写道:
>>> Hi Jia,
>>>
>>> LGTM
>>>
>>> -- Igor
>>>
>>>>> On Feb 6, 2020, at 1:20 AM, Jia Huang <huangjia at loongson.cn> wrote:
>>>> Hi all,
>>>>
>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8238586
>>>>
>>>> When TieredCompilation is disabled, the test will fail.
>>>> Please review the following fix which make the test pass when disabled TieredCompilation.
>>>> ------------------------------------------------------
>>>> diff -r e45d8ca836b5 -r 67bef3d6692f test/hotspot/jtreg/vmTestbase/jit/tiered/Test.java
>>>> --- a/test/hotspot/jtreg/vmTestbase/jit/tiered/Test.java  Mon Feb 03 22:10:59 2020 +0800
>>>> +++ b/test/hotspot/jtreg/vmTestbase/jit/tiered/Test.java  Thu Feb 06 15:25:19 2020 +0800
>>>> @@ -46,7 +46,7 @@
>>>>   import jdk.test.lib.process.ProcessTools;
>>>>
>>>>   public class Test {
>>>> -    private static String UNSUPPORTED_OPTION_MESSAGE = "-XX:TieredCompilation not supported in this VM";
>>>> +    private static String UNSUPPORTED_OPTION_MESSAGE = "-XX:+TieredCompilation not supported in this VM";
>>>>       private static String REGEXP = "^[0-9.]+: \\[compile level=\\d";
>>>>       public static void main(String[] args) throws Exception {
>>>>           {
>>>> @@ -56,7 +56,7 @@
>>>>                       "-XX:+PrintTieredEvents",
>>>>                       "-version");
>>>>               var output = new OutputAnalyzer(pb.start());
>>>> -            if (output.getStdout().contains(UNSUPPORTED_OPTION_MESSAGE)) {
>>>> +            if (output.getStderr().contains(UNSUPPORTED_OPTION_MESSAGE)) {
>>>>                   throw new SkippedException(UNSUPPORTED_OPTION_MESSAGE);
>>>>               }
>>>>               output.shouldHaveExitValue(0)
>>>> ------------------------------------------------------
>>>> For more info, please see the JBS.
>>>>
>>>> Thanks a lot
>>>>
>>>> Best regards,
>>>> Jia.
>>>>