[8u] Request for enhancement backport approval for "8168318 : PPC64: Use cmpldi instead of li/cmpld"

Rob McKenna rob.mckenna at oracle.com
Tue Dec 13 15:31:02 UTC 2016


Hi Bruno,

Unless I'm mistaken, neither of you are OCA signatories:

http://www.oracle.com/technetwork/community/oca-486395.html

We cannot accept this enhancement request until:

1) The requestor is a signatory
2) The webrev is hosted on cr.openjdk.java.net or shared over email as a
patch

    -Rob

On 02/12/16 05:42, Bruno Alexandre Rosa wrote:
> Hi,
> 
> Thanks to Rob for pointing to the adequate process documentation.
> 
> On behalf of Igor, I'm submitting an enhancement request that conforms to the process:
> 
> This changeset adds a new matching rule to the instruction selection phase (adlc) of the ppc vm.
> It consists of the exchange of the pair li/cmpld with cmpldi. 
> Jtreg hotspot tests didn't regress. 
> Got an 1% performance improvement when benchmarking neo4j.
> Since the patch is rather small and affects only the ppc vm, the risks are minimal.
> 
> Please, let us if we are breaking any jdk8u ground rules.
> 
> Regards,
> Bruno Rosa
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8168318
> Webrev: https://igorsnunes.github.io/openjdk/webrev/8168318/
> Review: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024809.html
> URL: http://hg.openjdk.java.net/jdk9/hs/hotspot/rev/622d3fe587f2 


More information about the jdk8u-dev mailing list