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

[Igor Ignatyev]

that was fast! thanks David.

-- Igor

> On Jan 24, 2019, at 11:50 PM, David Holmes <david.holmes at oracle.com> wrote:
> 
> 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