[15] RFR: 8242895: failed: sanity at src/hotspot/share/opto/escape.cpp:2361

[Tobias Hartmann]

+1

Best regards,
Tobias

On 20.07.20 22:05, Vladimir Kozlov wrote:
> Good.
> 
> Thanks,
> Vladimir
> 
> On 7/20/20 12:22 PM, Jamsheed C M wrote:
>> Hi Vladimir,
>>
>> Added both the tests
>>
>> http://cr.openjdk.java.net/~jcm/8242895/webrev_fix_EA.04/
>>
>> Best Regards,
>>
>> Jamsheed
>>
>> On 20/07/2020 23:31, Vladimir Kozlov wrote:
>>> I asked to have 2 different test methods to reproduce 2 cases separately.
>>> You can't mix them.
>>>
>>> Regards,
>>> Vladimir
>>>
>>>
>>> On 7/20/20 6:48 AM, Jamsheed C M wrote:
>>>> Hi Tobias,
>>>> On 20/07/2020 19:05, Tobias Hartmann wrote:
>>>>> Hi Jamsheed,
>>>>>
>>>>> On 20.07.20 15:30, Jamsheed C M wrote:
>>>>>> Revised webrev: http://cr.openjdk.java.net/~jcm/8242895/webrev_fix_EA.03/
>>>>> You don't need #ifdef ASSERT in escape.cpp:2252. Otherwise looks good to me! No new webrev
>>>>> required.
>>>>
>>>> Missed removing it. Thank you for the review.
>>>>
>>>> Best regards,
>>>>
>>>> Jamsheed
>>>>
>>>>>
>>>>> Best regards,
>>>>> Tobias