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

Jamsheed C M jamsheed.c.m at oracle.com
Mon Jul 20 19:22:11 UTC 2020


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


More information about the hotspot-compiler-dev mailing list