RFR: 8267773: PhaseStringOpts::int_stringSize doesn't handle min_jint correctly [v2]

Mon May 31 15:06:05 UTC 2021

Updated.

Thanks for contributing the test!

Best regards,
Nils

On 2021-05-31 18:13, Aleksey Shipilev wrote:
> On Mon, 31 May 2021 15:06:56 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
>> Nils Eliasson has updated the pull request incrementally with two additional commits since the last revision:
>>
>>   - Remove test
>>   - Add and fix test
> My OpenJDK handle is "shade".
>
> -------------
>
> PR: https://git.openjdk.java.net/jdk/pull/4255