RFR: JDK-8216437 : PPC64: Add intrinsic for GHASH algorithm [v6]

Martin Doerr mdoerr at openjdk.org
Thu Jan 9 14:49:41 UTC 2025


On Wed, 8 Jan 2025 20:48:33 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> (https://github.com/openjdk/jdk/blob/a641932427cbe8453130593355372837d70a098f/src/java.base/share/classes/com/sun/crypto/provider/GHASH.java#L281) In the java code, the 0 check is handled.
>
> The C2 compiler replaces this Java code by the intrinsic.

`len` is passed to the stub without null check: https://github.com/openjdk/jdk/blob/f9b11332eccd8a8ffb4128308f442b209d07a3b1/src/hotspot/share/opto/library_call.cpp#L7599
But I can see null checks in GHASH.java for all callers of `processBlocks`. So, your assertion should be fine.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20235#discussion_r1908942391


More information about the hotspot-dev mailing list