[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