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

Jamsheed C M jamsheed.c.m at oracle.com
Tue Jul 21 06:40:32 UTC 2020


Thank you for the reviews.

Initiated the Fix Request for JDK15, testing links are added in JBS.

Best regards,

Jamsheed

On 21/07/2020 11:47, Tobias Hartmann wrote:
> +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


More information about the hotspot-compiler-dev mailing list