> C1 has rudimentary conditional expression elimination and graph > canonicalization. However, those optimizations on If nodes currently fold > substitutability checks to pointer comparisons loosing value object semantics > for acmp. This PR fixes this. > > Testing: > - [x] tier1,tier2,tier3,valhalla-comp-stress with `--enable-preview` > > A lot of credit for this PR goes to @TobiHartmann who developed the inital > fix. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai).
Manuel Hässig has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains two new commits since the last revision: - Review - Fix JDK-8384871 ------------- Changes: - all: https://git.openjdk.org/valhalla/pull/2500/files - new: https://git.openjdk.org/valhalla/pull/2500/files/ee76c81f..547255c3 Webrevs: - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2500&range=01 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2500&range=00-01 Stats: 5 lines in 3 files changed: 1 ins; 1 del; 3 mod Patch: https://git.openjdk.org/valhalla/pull/2500.diff Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2500/head:pull/2500 PR: https://git.openjdk.org/valhalla/pull/2500
