RFR: 8275586: Zero: Simplify interpreter initialization

Aleksey Shipilev shade at openjdk.java.net
Wed Oct 20 07:54:22 UTC 2021


The prolog in `BytecodeInterpreter` is hairy due to early initialization of interpreter statics. Previous rewrites make it mostly redundant, and we can now simplify it.

This also implicitly fixes a initialization bug. If `JvmtiExport::can_post_interpreter_events()` changes at runtime, we will call into the uninitialized version:


    // Call the interpreter
    if (JvmtiExport::can_post_interpreter_events()) {
      BytecodeInterpreter::run<true>(istate);
    } else {
      BytecodeInterpreter::run<false>(istate);
    } 


Additional testing:
 - [ ] Linux x86_64 fastdebug `make bootcycle-images`

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

Commit messages:
 - Fix

Changes: https://git.openjdk.java.net/jdk/pull/6029/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6029&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8275586
  Stats: 70 lines in 3 files changed: 7 ins; 48 del; 15 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6029.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6029/head:pull/6029

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


More information about the hotspot-dev mailing list