RFR (XS): 8217371: C2: Incorrect LP64 guard in x86.ad after JDK-8210764 (Update avx512 implementation)

[Tobias Hartmann]

Hi Sandhya,

looks good to me too.

Best regards,
Tobias

On 28.01.19 09:40, Nils Eliasson wrote:
> Hi Sandhya,
> 
> Looks good,
> 
> Regards,
> 
> Nils
> 
> On 2019-01-27 04:47, Viswanathan, Sandhya wrote:
>>
>> Hi All,
>>
>>  
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8217371
>> <https://bugs.openjdk.java.net/browse/JDK-8217371>
>>
>> Webrev: http://cr.openjdk.java.net/~sviswanathan/8217371/webrev.00/
>> <http://cr.openjdk.java.net/~sviswanathan/8217371/webrev.00/>
>>
>>  
>>
>> The above webrev fixes the incorrect LP64 guard issue in x86.ad file.
>>
>> I have corrected the guard to _LP64 and updated the spill/fill instructions.  This bug only
>> affected the Knights family where AVX512VL is not supported.
>>
>>  
>>
>> I have tested it on SKX and Knights family with compiler jtreg tests.
>>
>>  
>>
>> Please review.
>>
>>  
>>
>> Best Regards,
>>
>> Sandhya
>>
>>  
>>
>>  
>>
>>  
>>