RFR: 8256730: Code that uses Object.checkIndex() range checks doesn't optimize well [v2]
Roland Westrelin
roland at openjdk.java.net
Fri Nov 20 13:35:24 UTC 2020
> This was reported by Paul with the vector API. There are 2 issues:
>
> - CastII nodes (added by Objects.checkIndex()) gets in the way of the
> pattern matching performed by range check elimination
>
> - By transforming (CastII (AddI x y)) into (AddI (CastII x) (CastII y))
> some CastII can be eliminated which improves address computation code.
Roland Westrelin 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 three additional commits since the last revision:
- build fix
- Merge branch 'master' into JDK-8256730
- step over CastII in range checks & push CastII thru add
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/1342/files
- new: https://git.openjdk.java.net/jdk/pull/1342/files/e70ea12b..76e6348d
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1342&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1342&range=00-01
Stats: 71 lines in 9 files changed: 51 ins; 7 del; 13 mod
Patch: https://git.openjdk.java.net/jdk/pull/1342.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/1342/head:pull/1342
PR: https://git.openjdk.java.net/jdk/pull/1342
More information about the hotspot-compiler-dev
mailing list