[9] RFR(S): 8080156: Integer.toString(int value) sometimes throws NPE

[Tobias Hartmann]

Hi Andreas,

thanks, I already noticed while running through JPRT and fixed it.

Best,
Tobias


On 22.05.2015 15:07, Andreas Eriksson wrote:
> Hi Tobias,
> 
> You have a comma that shouldn't be there in the bugid line:
> --------------------------------------------------
> TEST: compiler/types/TestTypePropagationToCmpU.java
> TEST RESULT: Error. Parse Exception: Invalid or unrecognized bugid: 8080156,
> --------------------------------------------------
> 
> Otherwise it looks good to me.
> Thanks for pushing this test.
> 
> - Andreas
> 
> On 2015-05-21 20:51, Tobias Hartmann wrote:
>> Hi,
>>
>> this is a follow up RFR on JDK-8060036 [1] that adds a regression test I used while investigating JDK-8080156 which turned out to be a duplicate.
>>
>> https://bugs.openjdk.java.net/browse/JDK-8080156
>> http://cr.openjdk.java.net/~thartmann/8080156/webrev.00/
>>
>> The test triggers a range check that is implemented with a CmpUNode. If type information is not propagated correctly to the CmpUNode, the range check trap is always taken. Because C2 also removes the unnecessary buffer allocation, a NullPointerException is thrown and the test fails.
>>
>> Thanks,
>> Tobias
>>
>> [1] http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2015-May/017983.html
>