> To re-enable the autobox cache, this patch reverts (parts of) > [JDK-8369921](https://bugs.openjdk.org/browse/JDK-8369921), > [JDK-8378476](https://bugs.openjdk.org/browse/JDK-8378476), deopt parts of > [JDK-8378531](https://bugs.openjdk.org/browse/JDK-8378531) and > [JDK-8379148](https://bugs.openjdk.org/browse/JDK-8379148). > > I had to modify the `newCacheArray` methods to return a non-flattened array > to avoid buffering. > > I wasn't sure about the comments above the `valueOf` methods: > https://github.com/openjdk/valhalla/commit/b3ccb6abfdb6e2bff59a7d79bf01b669ac319257#diff-6136f7a80110ec00bf9a0b7da9943eb2fc33025ab400bc1f70153c1b2ac36993R146 > > Should they be reverted too? For now, I left them as is. > > Thanks, > Tobias > > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai).
Tobias Hartmann has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision: - Merge branch 'lworld' into JDK-8383559 - Reviewer comments - Update src/hotspot/share/opto/type.cpp - JDK-8383559 ------------- Changes: - all: https://git.openjdk.org/valhalla/pull/2390/files - new: https://git.openjdk.org/valhalla/pull/2390/files/d51e0ad0..94f0f66e Webrevs: - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2390&range=03 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2390&range=02-03 Stats: 23482 lines in 661 files changed: 17215 ins; 3436 del; 2831 mod Patch: https://git.openjdk.org/valhalla/pull/2390.diff Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2390/head:pull/2390 PR: https://git.openjdk.org/valhalla/pull/2390
