> 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 updated the pull request incrementally with one additional 
commit since the last revision:

  Remove superfluous comments

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2500/files
  - new: https://git.openjdk.org/valhalla/pull/2500/files/547255c3..b6a7a5a9

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

  Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 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

Reply via email to