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