RFR(S): 8223140: Clean-up in 'ok_to_convert()'.

Patric Hedlin patric.hedlin at oracle.com
Thu May 2 13:32:58 UTC 2019


Thanks Nils.

/Patric

On 02/05/2019 14:41, Nils Eliasson wrote:
> +1
>
> Regards,
>
> Nils
>
>
> On 2019-04-30 18:54, Vladimir Ivanov wrote:
>> Looks good.
>>
>> I like precond/postcond macros.
>>
>> +static bool is_cloop_increment(Node* inc) {
>> +  precond(inc->Opcode() == Op_AddI || inc->Opcode() == Op_AddL);
>>
>> Best regards,
>> Vladimir Ivanov
>>
>> On 30/04/2019 07:11, Patric Hedlin wrote:
>>> Dear all,
>>>
>>> I would like to ask for help to review the following change/update:
>>>
>>> Issue:  https://bugs.openjdk.java.net/browse/JDK-8223140
>>> Webrev: http://cr.openjdk.java.net/~phedlin/tr8223140/
>>>
>>> 8223140: Clean-up in 'ok_to_convert()'
>>>
>>>      Simplify logic in 'ok_to_convert()'.
>>>      Rename 'is_loop_iv()' to 'is_cloop_ind_var()'.
>>>      Adding precond/postcond macros.
>>>
>>>
>>> Testing: Part of 8216137 (hs-tier1..4, hs-precheckin-comp, 
>>> Kitchensink24h)
>>>
>>>
>>> Best regards,
>>> Patric
>>>



More information about the hotspot-compiler-dev mailing list