RFR: 8308463: Refactor regenerated class handling in lambdaFormInvokers.cpp [v2]
Matias Saavedra Silva
matsaave at openjdk.org
Thu Jun 22 19:03:04 UTC 2023
On Thu, 22 Jun 2023 03:52:28 GMT, Ioi Lam <iklam at openjdk.org> wrote:
>> The handling of regenerated classes need to be refactored to enable future development:
>>
>> - Improve the API to access the regenerated classes and their methods
>> - Allow more classes to be regenerated, not just the Lambda Form invokers. Therefore, the code is moved to a new file cds/regeneratedClasses.cpp
>>
>>
>> Please see the bug report [JDK-8308463](https://bugs.openjdk.org/browse/JDK-8308463) for motivation of this PR.
>
> Ioi Lam has updated the pull request incrementally with two additional commits since the last revision:
>
> - @dholmes-ora comments
> - @coleenp review comments
Nice refactor! Just one question, otherwise it looks good.
src/hotspot/share/cds/regeneratedClasses.cpp line 61:
> 59: if (regen_m == nullptr) {
> 60: ResourceMark rm;
> 61: log_info(cds, lambda)("Method in original class is missing from regenerated class: " INTPTR_FORMAT " %s",
Should this result in some sort of exception?
-------------
Marked as reviewed by matsaave (Committer).
PR Review: https://git.openjdk.org/jdk/pull/14573#pullrequestreview-1493827980
PR Review Comment: https://git.openjdk.org/jdk/pull/14573#discussion_r1238920856
More information about the hotspot-runtime-dev
mailing list