RFR: 8275586: Zero: Simplify interpreter initialization
Aleksey Shipilev
shade at openjdk.java.net
Thu Nov 4 08:03:09 UTC 2021
On Wed, 20 Oct 2021 07:44:36 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> 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:
> - [x] Linux x86_64 fastdebug `make bootcycle-images`
I think I need a second (R)eviewer for this.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6029
More information about the hotspot-dev
mailing list