[14]: RFR(XXS): 8236944:The legVecZ operand should be limited to zmm0-zmm15 registers

[Viswanathan, Sandhya]

Pushed to http://hg.openjdk.java.net/jdk/jdk14.

Thanks,
Sandhya

-----Original Message-----
From: Tobias Hartmann <tobias.hartmann at oracle.com> 
Sent: Sunday, January 12, 2020 10:21 PM
To: Viswanathan, Sandhya <sandhya.viswanathan at intel.com>; Vladimir Ivanov <vladimir.x.ivanov at oracle.com>; hotspot compiler <hotspot-compiler-dev at openjdk.java.net>
Subject: Re: [14]: RFR(XXS): 8236944:The legVecZ operand should be limited to zmm0-zmm15 registers

Hi Sandhya,

looks good to me too. Please push to
http://hg.openjdk.java.net/jdk/jdk14

Thanks,
Tobias

On 11.01.20 01:25, Viswanathan, Sandhya wrote:
> Thanks a lot Vladimir! Can I push this onto JDK 14 branch?
> 
> Best Regards,
> Sandhya
> 
> 
> -----Original Message-----
> From: Vladimir Ivanov <vladimir.x.ivanov at oracle.com> 
> Sent: Friday, January 10, 2020 3:39 PM
> To: Viswanathan, Sandhya <sandhya.viswanathan at intel.com>; hotspot compiler <hotspot-compiler-dev at openjdk.java.net>
> Subject: Re: [14]: RFR(XXS): 8236944:The legVecZ operand should be limited to zmm0-zmm15 registers
> 
> FYI test results are clean (hs-precheckin-comp,hs-tier1,hs-tier2).
> 
> Best regards,
> Vladimir Ivanov
> 
> On 11.01.2020 00:50, Vladimir Ivanov wrote:
>> Looks good.
>>
>> Best regards,
>> Vladimir Ivanov
>>
>> On 10.01.2020 22:35, Viswanathan, Sandhya wrote:
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8236944
>>>
>>> Webrev: http://cr.openjdk.java.net/~sviswanathan/8236944/webrev.00/
>>>
>>> Currently legVecZ operand is defined as follows:
>>>
>>> operand legVecZ() %{
>>>
>>>    constraint(ALLOC_IN_RC(vectorz_reg_vl));
>>>
>>>    match(VecZ);
>>>
>>>    format %{ %}
>>>
>>>    interface(REG_INTER);
>>>
>>> %}
>>>
>>> This allows legVecZ to be zmm0-zmm31 on platforms supporting AVX 512 vl.
>>>
>>> The legVecZ should instead be defined as:
>>>
>>> operand legVecZ() %{
>>>
>>>    constraint(ALLOC_IN_RC(vectorz_reg_legacy));
>>>
>>>    match(VecZ);
>>>
>>>    format %{ %}
>>>
>>>    interface(REG_INTER);
>>>
>>> %}
>>>
>>> Please consider this for JDK 14.
>>>
>>> Best Regards,
>>>
>>> Sandhya
>>>