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