Integrated: 8265911: assert(comp != __null) failed: Compiler instance missing

Frank King github.com+16741972+whjpji at openjdk.java.net
Mon Apr 26 11:10:40 UTC 2021


On Sun, 25 Apr 2021 08:56:39 GMT, Frank King <github.com+16741972+whjpji at openjdk.org> wrote:

> As I debugged into the code, the `_c1_count` was set to `1` [1] with flags `-XX:+EnableJVMCI -XX:+UseJVMCICompiler -XX:JVMCIHostThreads=1`. However, [2] would never be executed, and thus `_compilers[0]` was `NULL`. 
> 
> When C1 is disabled, `_c1_count` should always be `0`, so I added this `#ifdef` guard to avoid changing value of `_c1_count` under this circumstance.
> 
> [1] https://github.com/openjdk/jdk/blob/f1f2afda5a0ab8f213e8a1b5324a251928c8d81a/src/hotspot/share/compiler/compileBroker.cpp#L656
> [2] https://github.com/openjdk/jdk/blob/f1f2afda5a0ab8f213e8a1b5324a251928c8d81a/src/hotspot/share/compiler/compileBroker.cpp#L664

This pull request has now been integrated.

Changeset: 1580a473
Author:    Junji Wang <junjiwang at tencent.com>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/1580a473
Stats:     2 lines in 1 file changed: 2 ins; 0 del; 0 mod

8265911: assert(comp != __null) failed: Compiler instance missing

Reviewed-by: kvn, thartmann

-------------

PR: https://git.openjdk.java.net/jdk/pull/3672


More information about the hotspot-compiler-dev mailing list