RFR: 8317269: Store old classes in linked state in AOT cache [v2]

Ioi Lam iklam at openjdk.org
Wed Aug 27 23:31:47 UTC 2025


On Tue, 19 Aug 2025 23:29:46 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> I don't know why you don't fix these together, ie why does this differentiate < 50 classfiles or 50 that failed over to the old verifier?  Seems like is_old_class_for_verifier should be both.  You might have to set a status flag in the classfile for this though.
>
> This PR is complicated enough, so I want to fix the `== 50` case in a separate PR.

> I will add a test case in this PR to make sure this case is covered.

Actually this case is already covered by an existing test that checks for the `"Skipping ...: Verified with old verifier"` log:

https://github.com/openjdk/jdk/blob/c5cbcac828e1c7aa845cf16e68f6306ae49e050c/test/hotspot/jtreg/runtime/cds/appcds/aotCache/VerifierFailOver.java#L45-L47

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26754#discussion_r2305555249


More information about the hotspot-dev mailing list