RFR(XXXS): 8208706 compiler/tiered/ConstantGettersTransitionsTest.java fails to compile

[Daniel D. Daugherty]

Thanks!

Dan


On 8/2/18 10:09 PM, David Holmes wrote:
> Reviewed.
>
> Thanks,
> David
>
> On 3/08/2018 12:06 PM, Daniel D. Daugherty wrote:
>> Greetings,
>>
>> One more tweak needed to the fix for:
>>
>>      JDK-8208655 use JTreg skipped status in hotspot tests
>>      https://bugs.openjdk.java.net/browse/JDK-8208655
>>
>> I am looking for a single (R)eviewer for this trivial fix.
>>
>> This one is tracked by the following:
>>
>>      JDK-8208706 compiler/tiered/ConstantGettersTransitionsTest.java 
>> fails to compile
>>      https://bugs.openjdk.java.net/browse/JDK-8208706
>>
>> The fix is simple:
>>
>> $ hg diff
>> diff -r 531de4cbaa4f 
>> test/hotspot/jtreg/compiler/tiered/ConstantGettersTransitionsTest.java
>> --- 
>> a/test/hotspot/jtreg/compiler/tiered/ConstantGettersTransitionsTest.java 
>> Thu Aug 02 16:16:02 2018 -0700
>> +++ 
>> b/test/hotspot/jtreg/compiler/tiered/ConstantGettersTransitionsTest.java 
>> Thu Aug 02 22:02:04 2018 -0400
>> @@ -40,6 +40,7 @@
>>   package compiler.tiered;
>>
>>   import compiler.whitebox.CompilerWhiteBoxTest;
>> +import jtreg.SkippedException;
>>
>>   import java.lang.reflect.Executable;
>>   import java.util.concurrent.Callable;
>>
>>
>> I verified that the test failed to compile via:
>>
>>      make CONF=linux-x86_64-normal-server-release run-test 
>> TEST=open/test/hotspot/jtreg/compiler/tiered/ConstantGettersTransitionsTest.java 
>>
>>
>> on my Linux server, applied the change and verified that
>> the test now compiles and passes.
>>
>> Thanks, in advance, for any questions, comments or suggestions.
>>
>> Dan
>>