RFR: 8331429: [JVMCI] Cleanup JVMCIRuntime allocation routines

Dean Long dlong at openjdk.org
Sat May 11 08:07:02 UTC 2024


On Sat, 11 May 2024 07:41:17 GMT, Doug Simon <dnsimon at openjdk.org> wrote:

>> src/hotspot/share/jvmci/jvmciRuntime.cpp line 131:
>> 
>>> 129:       // Cannot re-execute class initialization without side effects
>>> 130:       // so return without attempting the initialization
>>> 131:       return;
>> 
>> Do we need to call `current->set_vm_result(nullptr)` on these bailout paths?
>
> That's done in `~RetryableAllocationMark`.

Only for the HAS_PENDING_EXCEPTION case.  What about the !h->is_initialized() case?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19176#discussion_r1597394595


More information about the graal-dev mailing list