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