[11u backport] RFR(S): 8215317: [GRAAL] unit test CheckGraalIntrinsics failed after 8213754

[Gustavo Romero]

On 01/18/2019 06:26 PM, Vladimir Kozlov wrote:
> Looks good.

Thanks for the review, Vladimir!

Regards,
Gustavo

> Thanks,
> Vladimir
> 
> On 1/18/19 6:57 AM, Gustavo Romero wrote:
>> Hi,
>>
>> Could the following backport to 11u be reviewed, please?
>>
>> Bug     : https://bugs.openjdk.java.net/browse/JDK-8215317
>> Change  : http://hg.openjdk.java.net/jdk/jdk/rev/108a161aed93
>> Backport: http://cr.openjdk.java.net/~gromero/8215317_jdk11u/v1/
>>
>> It adds 4 intrinsics to the Graal test CheckGraalIntrinsics.java list so
>> JDK 11u becomes aware of them. Otherwise that test will break once change
>> 8213754 [0] lands 11u (which will effectively add the 4 intrinsics to
>> PPC64/Hotspot and adapt the correlated methods to be intrinsified).
>>
>> The backport changed the inclusion of the intrinsics for JDK 11 or higher,
>> instead for JDK 12 or higher (original patch).
>>
>> This backport was tested on x86_64 with
>> ./test/hotspot/jtreg/compiler/{c1,c2,intrinsics} plus
>> ./test/hotspot/jtreg/compiler/graalunit (with Graal compiler enabled)
>> and no regressions were observed too.
>>
>> Thank you.
>>
>> Best regards,
>> Gustavo
>>
>> [0] https://bugs.openjdk.java.net/browse/JDK-8213754
>>
>