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

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Mon Jan 21 10:54:21 UTC 2019


Hi,

looks good to me.

Best regards,
  Goetz.

> -----Original Message-----
> From: Gustavo Romero <gromero at linux.vnet.ibm.com>
> Sent: Freitag, 18. Januar 2019 15:57
> To: hotspot-compiler-dev at openjdk.java.net; Lindenmaier, Goetz
> <goetz.lindenmaier at sap.com>; vladimir.kozlov at oracle.com
> Subject: [11u backport] RFR(S): 8215317: [GRAAL] unit test
> CheckGraalIntrinsics failed after 8213754
> 
> 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