[lworld] RFR: 8262128: [lworld] C1's ValueNumbering optimization does not correctly handle delayed accesses [v2]

Tobias Hartmann thartmann at openjdk.java.net
Mon Feb 22 15:41:12 UTC 2021


> C1's ValueNumbering optimization replaces an array+field load `array[0].intArray` by another non-equivalent array+field load `array[0].floatArray` because it does not handle delayed accesses. The hash value of `LoadIndexed` needs to take delayed accesses into account.
> 
> Best regards,
> Tobias

Tobias Hartmann has updated the pull request incrementally with one additional commit since the last revision:

  Disable ValueNumbering for now

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

Changes:
  - all: https://git.openjdk.java.net/valhalla/pull/345/files
  - new: https://git.openjdk.java.net/valhalla/pull/345/files/3363401b..f17af09f

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=valhalla&pr=345&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=valhalla&pr=345&range=00-01

  Stats: 11 lines in 2 files changed: 0 ins; 10 del; 1 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/345.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/345/head:pull/345

PR: https://git.openjdk.java.net/valhalla/pull/345



More information about the valhalla-dev mailing list