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

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Jan 18 20:26:19 UTC 2019


Looks good.

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
> 


More information about the hotspot-compiler-dev mailing list