RFR(T) [12] : 8217770 : problem list org.graalvm.compiler.debug.test.DebugContextTest

[David Holmes]

Looks good.

Thanks,
David

On 25/01/2019 5:46 pm, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev//8217770/webrev.00/index.html
>> 1 line changed: 1 ins; 0 del; 0 mod;
> 
>> diff -r 6533b2b34593 test/hotspot/jtreg/ProblemList-graal.txt
>>   org.graalvm.compiler.core.test.OptionsVerifierTest                               8205081
>>   org.graalvm.compiler.hotspot.test.CompilationWrapperTest                         8205081
>>   org.graalvm.compiler.replacements.test.classfile.ClassfileBytecodeProviderTest   8205081
>> +org.graalvm.compiler.debug.test.DebugContextTest                                 8205081
>>   
>>   org.graalvm.compiler.core.test.deopt.CompiledMethodTest          8202955
> 
> Hi all,
> 
> could you please review this tiny and trivial patch which puts org.graalvm.compiler.debug.test.DebugContextTest back into the problem list?
> 
> this graal unit test was prematurely removed from the problem list by 8217580[1-2]. this test is/was known to fail not only because of 8203504[3], but also because of 8205081[4]. this patch put DebugContextTest test to the problem list w/ 8205081 as the reason.
> 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8217770
> webrev: http://cr.openjdk.java.net/~iignatyev//8217770/webrev.00/index.html
> testing: compiler/graalunit/DebugTest.java (which includes this unit test)
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8217580
> [2] http://hg.openjdk.java.net/jdk/jdk12/rev/6533b2b34593#l2.44
> [3] https://bugs.openjdk.java.net/browse/JDK-8203504
> [4] https://bugs.openjdk.java.net/browse/JDK-8205081
> 
> Thanks,
> -- Igor
>