compiler/jvmci/compilerToVM/HasCompiledCodeForOSRTest.java fails with Graal

[Christian Thalinger]

> On Aug 11, 2017, at 12:09 PM, Christian Thalinger <cthalinger at twitter.com> wrote:
> 
>> 
>> On Aug 11, 2017, at 11:47 AM, Doug Simon <doug.simon at oracle.com> wrote:
>> 
>> 
>>> On 11 Aug 2017, at 11:16, Christian Thalinger <cthalinger at twitter.com> wrote:
>>> 
>>>> 
>>>> On Jun 27, 2017, at 7:20 PM, Doug Simon <doug.simon at oracle.com> wrote:
>>>> 
>>>> 
>>>>> On 27 Jun 2017, at 18:34, Christian Thalinger <cthalinger at twitter.com> wrote:
>>>>> 
>>>>>> 
>>>>>> On Jun 26, 2017, at 8:47 PM, Doug Simon <doug.simon at oracle.com> wrote:
>>>>>> 
>>>>>> 
>>>>>>> On 27 Jun 2017, at 01:37, Christian Thalinger <cthalinger at twitter.com> wrote:
>>>>>>> 
>>>>>>> So, this change:
>>>>>>> 
>>>>>>> $ hg log -r fbb9c8026495 -p compiler/jvmci/compilerToVM/HasCompiledCodeForOSRTest.java
>>>>>>> changeset:   12748:fbb9c8026495
>>>>>>> tag:         jdk-9+168
>>>>>>> user:        dnsimon
>>>>>>> date:        Fri Apr 28 02:54:05 2017 -0700
>>>>>>> summary:     8177845: Need a mechanism to load Graal
>>>>>>> 
>>>>>>> diff --git a/test/compiler/jvmci/compilerToVM/HasCompiledCodeForOSRTest.java b/test/compiler/jvmci/compilerToVM/HasCompiledCodeForOSRTest.java
>>>>>>> --- a/test/compiler/jvmci/compilerToVM/HasCompiledCodeForOSRTest.java
>>>>>>> +++ b/test/compiler/jvmci/compilerToVM/HasCompiledCodeForOSRTest.java
>>>>>>> @@ -39,7 +39,7 @@
>>>>>>> * @run main/othervm -Xbootclasspath/a:.
>>>>>>> *                   -XX:+UnlockDiagnosticVMOptions -XX:+WhiteBoxAPI
>>>>>>> *                   -XX:+UnlockExperimentalVMOptions -XX:+EnableJVMCI
>>>>>>> - *                   -XX:-BackgroundCompilation
>>>>>>> + *                   -XX:-BackgroundCompilation -Djvmci.Compiler=null
>>>>>>> *                   compiler.jvmci.compilerToVM.HasCompiledCodeForOSRTest
>>>>>>> */
>>>>>>> 
>>>>>>> 
>>>>>>> sets the JVMCI compiler to null.  How is this test supposed to work when running with UseJVMCICompiler and Graal?
>>>>>> 
>>>>>> It's not which is why the flags needed to enable Graal aren't present.
>>>>> 
>>>>> But what if you run with -vmoptions:”-XX:+UnlockExperimentalVMOptions -XX:+EnableJVMCI -XX:+UseJVMCICompiler”?  Which is what you do when you want to run all tests with Graal.
>>>> 
>>>> Then I guess the test will have to be modified/fixed.
>>> 
>>> Has anyone filed a bug and/or picked this up?
>> 
>> Not as far as I'm aware.
> 
> I’ll file one.

https://bugs.openjdk.java.net/browse/JDK-8186136
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20170811/9fc863a1/attachment.html>