RFR: 8154495: SHA256 AVX2 intrinsic (when no supports_sha() available)
Vladimir Kozlov
vladimir.kozlov at oracle.com
Tue Apr 19 00:09:10 UTC 2016
Hi Jan,
The patch was generated on Windows and have ^M at the end of lines so I
can't apply it to our sources.
I don't see usage of new [v]movdqa(), vpsrldq(), vpslldq(), instructions.
Please, move new code in macroAssembler_x86_sha.cpp to the end of file.
_k256_W[] is the same as _k256[] with repeated 4 values. I would suggest
to generated it dynamically in stubGenerator_x86_64.cpp based on _k256:
StubRoutines::x86::_k256_W_adr = generate_k256_W();
What testing was done? Did you ran with fastdebug build? I am concern
about size of new stub and current code_size2 is enough.
Thanks,
Vladimir
On 4/18/16 2:44 PM, Civlin, Jan wrote:
> == Correction in the subject line ===
>
> We would like to contribute the SHA256 AVX2 intrinsic.
>
> This intrinsic is for x86 AVX2 architecture when no supports_sha() is available. It is L64 code only.
>
> The patch delivers x2 performance gain for the latency and throughput - both are measured on an average message.
>
> Contributor: Jan Civlin.
>
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8154495
> webrev: http://cr.openjdk.java.net/~vdeshpande/8154495/webrev.00/
>
More information about the hotspot-compiler-dev
mailing list