RFR: C2 changes to support Java Object Monitors with Interpreter

Dean Long dlong at openjdk.org
Fri Oct 13 21:46:29 UTC 2023


On Thu, 12 Oct 2023 15:46:29 GMT, Ahmed Muhsin <duke at openjdk.org> wrote:

> Only known issue is with the ProfileInterpreter on however, we implemented a work around: 9bca6603204e94ffac4be450b73610cbaf9eed0f 
> 
> Testing:
> Compared jtreg results from jom-master to our PR with ProfileInterpreter off and with ProfileInterpreter on with the work around. Note that with ProfileInterpreter on and no work around jtreg errors out (so that configuration is not included in the test tables below).
> 
> See test results in comments.

src/hotspot/share/opto/parse1.cpp line 1131:

> 1129:         if (ObjectMonitorMode::java()) {
> 1130:           if (!ex_jvms->has_method()){
> 1131:             ex_jvms->set_method(method());

We probably don't need this if we find out why the caller state doesn't have a method.  I'll see if I can find the problem.

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

PR Review Comment: https://git.openjdk.org/loom/pull/206#discussion_r1358873255


More information about the loom-dev mailing list