RFR (M) 8175368: [s390] Provide intrinsic implementation for CRC32C

Schmidt, Lutz lutz.schmidt at sap.com
Mon Feb 27 20:38:39 UTC 2017


Hi all,
thanks, Vladimir, for pointing me to this. I already noticed it, corrected the URL and sent an update to the DL (approx. 3h ago). 
Regards,
Lutz


On 27.02.17, 21:31, "Vladimir Kozlov" <vladimir.kozlov at oracle.com> wrote:

    Hi, Lutz
    
    The link to webrev is wrong. It points to 8175370 changes.
    
    Vladimir
    
    On 2/27/17 6:55 AM, Schmidt, Lutz wrote:
    > Hi all,
    >
    >
    >
    > may I kindly request reviews for my medium enhancement? Further down the
    > road, I would need a sponsor, too.
    >
    >
    >
    > Bug: https://bugs.openjdk.java.net/browse/JDK-8175368
    >
    > Webrev: http://cr.openjdk.java.net/~lucy/webrevs/8175368/
    > <http://cr.openjdk.java.net/~lucy/webrevs/8175370/>
    >
    >
    >
    > Description:
    >
    > This intrinsic implementation provides some performance benefit over the
    > standard Java implementation. It uses only well-known “standard”
    > instructions, available on all supported IBM System z hardware. Being
    > very similar to the CRC32 intrinsics, it was tried to share as much code
    > as possible between these two.
    >
    >
    >
    > Performance may be expected to increase up to 2.0x, compared to a run
    > with –XX:-UseCRC32CIntrinsics. Actual speedup depends on the length of
    > the byte array fed into CRC32C and, to some extent, on the H/W
    > generation the test runs on.
    >
    >
    >
    > Thanks,
    >
    > Lutz
    >
    >
    >
    



More information about the hotspot-compiler-dev mailing list