RFR: 8267773: PhaseStringOpts::int_stringSize doesn't handle min_jint correctly [v2]
Nils Eliasson
neliasso at openjdk.java.net
Mon May 31 15:06:56 UTC 2021
> Hi,
>
> PhaseStringOpts::int_stringSize doesn't handle min_jint correctly - it can't be negated to a positive number. I fix this adding a special case for this.
>
> Added a test that was contributed by Adam who reported the bug. Will add as a contributor.
>
> Please review,
> Best regards,
> Nils Eliasson
Nils Eliasson has updated the pull request incrementally with two additional commits since the last revision:
- Remove test
- Add and fix test
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/4255/files
- new: https://git.openjdk.java.net/jdk/pull/4255/files/da17bb36..43125189
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4255&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4255&range=00-01
Stats: 130 lines in 2 files changed: 60 ins; 70 del; 0 mod
Patch: https://git.openjdk.java.net/jdk/pull/4255.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/4255/head:pull/4255
PR: https://git.openjdk.java.net/jdk/pull/4255
More information about the hotspot-compiler-dev
mailing list