> 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.
Paul Hübner has updated the pull request incrementally with one additional commit since the last revision: Reviewer feedback. ------------- Changes: - all: https://git.openjdk.org/valhalla/pull/2249/files - new: https://git.openjdk.org/valhalla/pull/2249/files/ed2acbd7..d7197572 Webrevs: - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2249&range=02 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2249&range=01-02 Stats: 34 lines in 6 files changed: 0 ins; 21 del; 13 mod Patch: https://git.openjdk.org/valhalla/pull/2249.diff Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2249/head:pull/2249 PR: https://git.openjdk.org/valhalla/pull/2249
