RFR: 8293187: Store initialized Enum classes in AOTCache [v10]
Ioi Lam
iklam at openjdk.org
Wed Oct 2 02:58:36 UTC 2024
On Tue, 1 Oct 2024 09:48:00 GMT, Andrew Dinn <adinn at openjdk.org> wrote:
>> I added a comment according to your suggestion. I'll also see if it's possible to require all supertypes to be aot-initialized. I think that will make the model simpler.
>
> Yes, it should be simpler, although perhaps at the cost of omitting some mirrors from the heap section. Anyway, the comment is helpful.
@adinn I added the enforcement that for an aot-inited class, all of its super types's <clinit> must also be executed during assembly phase. I think this model is easier to understand. It also forces you to check if the side effects of the supertypes' <clinit>s can also be cached. Please re-review this change: [ba6f280](https://github.com/openjdk/jdk/pull/20958/commits/ba6f2805875beb02298d1615674147b65c98a039)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20958#discussion_r1783783686
More information about the hotspot-dev
mailing list