RFR(XS): 8145333: -XX:+EnableJVMCI -XX:+UseJVMCICompiler -XX:-EnableJVMCI makes JVM crash

[Christian Thalinger]

Well, now you have to manually check all flags that had a constraint directive.  That is the annoying part and what I complained about in:

https://bugs.openjdk.java.net/browse/JDK-8145357 <https://bugs.openjdk.java.net/browse/JDK-8145357>

Also, commandLineFlagConstraintsJVMCI.{cpp,hpp} files are now empty and I think we should remove them.

> On Feb 11, 2016, at 9:47 PM, Jamsheed C m <jamsheed.c.m at oracle.com> wrote:
> 
> Hi Chris,
> 
> revised webrev: http://cr.openjdk.java.net/~thartmann/8145333/webrev.01/ <http://cr.openjdk.java.net/~thartmann/8145333/webrev.01/>
> 
> Best Regards,
> Jamsheed
> 
> On 2/11/2016 11:31 PM, Christian Thalinger wrote:
>> 
>>> On Feb 11, 2016, at 3:47 AM, Jamsheed C m < <mailto:jamsheed.c.m at oracle.com>jamsheed.c.m at oracle.com <mailto:jamsheed.c.m at oracle.com>> wrote:
>>> 
>>> Hi,
>>> 
>>> Request for review
>>> 
>>> bug url:  https://bugs.openjdk.java.net/browse/JDK-8145333 <https://bugs.openjdk.java.net/browse/JDK-8145333>
>>> web url: http://cr.openjdk.java.net/~thartmann/8145333/webrev.00/ <http://cr.openjdk.java.net/%7Ethartmann/8145333/webrev.00/>
>> 
>> That looks alright but we should remove the constraints from all the JVMCI command line flags because the way we use them is not supported.
>> 
>> Also, can you change the error message?  Right now it looks like this:
>> 
>> $ ./build/macosx-x86_64-normal-server-release/jdk/bin/java -XX:+UnlockExperimentalVMOptions -XX:+UseJVMCICompiler
>> EnableJVMCI must be enabled
>> Improperly specified VM option 'UseJVMCICompiler'
>> Error: Could not create the Java Virtual Machine.
>> Error: A fatal exception has occurred. Program will exit.
>> 
>> I think with your changes we will only see the first line without mentioning UseJVMCICompiler.  Is that correct?
>> 
>>> 
>>> Fix Summary: added jvmci flag consistency check after arg parsing.
>>> 
>>> tests: jprt
>>> unit test: command line verification
>>> 
>>> Best Regards,
>>> Jamsheed
>> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160212/f216a9cc/attachment.html>