RFR: 8278479: RunThese test failure with +UseHeavyMonitors and +VerifyHeavyMonitors

Dean Long dlong at openjdk.org
Fri Jul 1 20:22:47 UTC 2022


On Thu, 30 Jun 2022 22:05:14 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> This change adds a null check before calling into Runtime1::monitorenter when -XX:+UseHeavyMonitors is set. There's a null check in the C2 and interpreter code before calling the runtime function but not C1.
> 
> Tested with tier1-7 (a little of 8) and built on most non-oracle platforms as well.

Marked as reviewed by dlong (Reviewer).

src/hotspot/cpu/aarch64/c1_LIRAssembler_aarch64.cpp line 2563:

> 2561:     if (op->info() != NULL) {
> 2562:       int null_check_offset = __ offset();
> 2563:       __ null_check(obj, -1);

Suggestion:

      __ null_check(obj);

src/hotspot/cpu/arm/c1_LIRAssembler_arm.cpp line 2438:

> 2436:       int null_check_offset = __ offset();
> 2437:       __ null_check(obj);
> 2438:       add_debug_info_for_null_check(null_check_offset, op->info());

Is this equivalent to the following?

add_debug_info_for_null_check_here(op->info());
__ null_check(obj);

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

PR: https://git.openjdk.org/jdk/pull/9339


More information about the hotspot-compiler-dev mailing list