[11u backport] RFR(M): 8213754: PPC64: Add Intrinsics for isDigit/isLowerCase/isUpperCase/isWhitespace
martin.doerr at sap.com
Fri Jan 18 16:07:53 UTC 2019
hotspot part looks good, too.
From: Roger Riggs <Roger.Riggs at oracle.com>
Sent: Freitag, 18. Januar 2019 16:35
To: Gustavo Romero <gromero at linux.vnet.ibm.com>; hotspot-compiler-dev at openjdk.java.net; Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; Doerr, Martin <martin.doerr at sap.com>; vladimir.kozlov at oracle.com
Cc: Michihiro Horie <HORIE at jp.ibm.com>
Subject: Re: [11u backport] RFR(M): 8213754: PPC64: Add Intrinsics for isDigit/isLowerCase/isUpperCase/isWhitespace
Looks good for the jdk files.
On 01/18/2019 10:07 AM, Gustavo Romero wrote:
Could the following backport to 11u be reviewed, please?
Bug : https://bugs.openjdk.java.net/browse/JDK-8213754
Change : http://hg.openjdk.java.net/jdk/jdk/rev/7384e00d5860
It adds 4 intrinsics that use instructions introduced by POWER9 in order to
speed up methods isDigit, isLowerCase, isUpperCase, and isWhitespace.
The change is mostly PPC64-only but it does touch shared code, for
instance, in order to adapt the methods in question to be properly
intrinsified. It also needs an additional change , since one Graal
test has to be adapted (a separated RFR to backport  was sent to ).
The change applies almost cleanly: only a small tweak is necessary because
the hunk for ppc.ad file relies on some absent text in the 11u code around
the change to be applied. That absent text is related to the Superword
feature (a non-related feature), which is not backported yet to 11u.
This backport was tested on POWER8 and POWER9 and no regressions were
This backport was also tested on x86_64 with
./test/hotspot/jtreg/compiler/graalunit (with Graal compiler enabled) with
change 8215317  applied and no regressions were observed too.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-compiler-dev