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.

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

Commit messages:
 - Harden CDS dumping with HotSpot assertion.
 - Reviewer feedback on Java code cleanup.
 - Merge branch 'lworld' into JDK-8379148
 - Clean up autobox check coce.
 - Skip JVMCI autobox cache if preview is enabled.
 - Merge branch 'lworld' into JDK-8379148
 - Re-commit Axel's patch.

Changes: https://git.openjdk.org/valhalla/pull/2249/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=2249&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8379148
  Stats: 112 lines in 7 files changed: 52 ins; 0 del; 60 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

Reply via email to