Integrated: 8267773: PhaseStringOpts::int_stringSize doesn't handle min_jint correctly

Nils Eliasson neliasso at openjdk.java.net
Wed Jun 2 08:00:42 UTC 2021


On Fri, 28 May 2021 20:56:54 GMT, Nils Eliasson <neliasso at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: d47a77d2
Author:    Nils Eliasson <neliasso at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/d47a77d2d54ee8c41ef969b42b3336a646dffd7b
Stats:     65 lines in 2 files changed: 65 ins; 0 del; 0 mod

8267773: PhaseStringOpts::int_stringSize doesn't handle min_jint correctly

Co-authored-by: Adam Farley <afarley at openjdk.org>
Co-authored-by: Aleksey Shipilev <shade at openjdk.org>
Co-authored-by: Nils Eliasson <neliasso at openjdk.org>
Reviewed-by: roland

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

PR: https://git.openjdk.java.net/jdk/pull/4255


More information about the hotspot-compiler-dev mailing list