On Fri, 20 Mar 2026 13:06:37 GMT, Paul Hübner <[email protected]> wrote:
> Hi all, > > This change eliminates more box cache usages when running with > `--enable-preview`: > * The majority of the change is contributed by @xmas92 who disabled CDS > loading an archived cache. > * There's an autobox cache cast in C2 that needed to be disabled. > > Notably, this change does not prevent the `XXXCache`'s `cache` variables from > being CDS archived. This is due to the fact that the infrastructure is common > to preview and non-preview mode, and the code complexity is not justifiable > when the cost is storing five `null`s. Instead, it is ensured that the cache > is never read from. > > Testing: tiers 1-4. This pull request has now been integrated. Changeset: 897f6db9 Author: Paul Hübner <[email protected]> URL: https://git.openjdk.org/valhalla/commit/897f6db96c545ed5af170c1728f5a7f2835d6c94 Stats: 82 lines in 6 files changed: 35 ins; 8 del; 39 mod 8379148: [lworld] Wrapper class caches should not be used when --enable-preview Co-authored-by: Axel Boldt-Christmas <[email protected]> Reviewed-by: matsaave, thartmann, liach, iklam ------------- PR: https://git.openjdk.org/valhalla/pull/2249
