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

[Vladimir Kozlov]

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