> 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:

  Move autobox check to ciField.

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2249/files
  - new: https://git.openjdk.org/valhalla/pull/2249/files/04416544..ed2acbd7

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2249&range=01
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2249&range=00-01

  Stats: 6 lines in 2 files changed: 2 ins; 2 del; 2 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