RFR(XS): 8250233: -XX:+CITime triggers guarantee(events != NULL) in jvmci.cpp

[Vladimir Kozlov]

Nope, the check is correct. In hosted mode Graal is used as Java application and not as JIT compiler so that 
UseJVMCICompiler flag is false.

The problem is really caused by recent 8248321 changes - it is regression. Doug, please advice how to fix it.
I think the new JVMCI events code should be adjusted for hosted mode.

Thanks,
Vladimir

On 7/24/20 5:53 AM, Schmidt, Lutz wrote:
> Resending after updating subject line with bug id.
> Sorry for the spam.
> Lutz
> 
> On 24.07.20, 14:51, "Schmidt, Lutz" <lutz.schmidt at sap.com> wrote:
> 
>      Dear all,
> 
>      may I please request reviews for this small fix? I would even say it is a trivial fix. It inverts an if condition such that JVMCI specific code is called only when JVMCI compilation is enabled via UseJVMCICompiler.
> 
>      Bug:    https://bugs.openjdk.java.net/browse/JDK-8250233
>      Webrev: http://cr.openjdk.java.net/~lucy/webrevs/8250233.00/
> 
>      Local testing looks good. jdk/submit tests pending.
> 
>      Thank you!
>      Lutz
> 
> 
> 
>