RFR: 8252204: AArch64: Implement SHA3 accelerator/intrinsic [v10]

Fei Yang fyang at openjdk.java.net
Thu Oct 22 00:49:12 UTC 2020


On Wed, 21 Oct 2020 19:20:28 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> OK.  Will update with the following change after Aleksey's PR is integrated:
>> 
>> --- a/src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.hotspot.test/src/org/graalvm/compiler/hotspot/test/CheckGraalIntrinsics.java
>> +++ b/src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.hotspot.test/src/org/graalvm/compiler/hotspot/test/CheckGraalIntrinsics.java
>> @@ -608,6 +608,10 @@ public class CheckGraalIntrinsics extends GraalTest {
>>          if (!config.useSHA512Intrinsics()) {
>>              add(ignore, "sun/security/provider/SHA5." + shaCompressName + "([BI)V");
>>          }
>> +
>> +        if (isJDK16OrHigher()) {
>> +            add(toBeInvestigated, "sun/security/provider/SHA3." + shaCompressName + "([BI)V");
>> +        }
>>      }
>
> Yes, please, do that.

Done.
Commit: https://github.com/openjdk/jdk/pull/207/commits/b43f91970d44e6e0c1b3b4ef452ec388ecbecb83
I think this will not conflict with Aleksey's PR as we modify in different places of CheckGraalIntrinsics.java

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

PR: https://git.openjdk.java.net/jdk/pull/207



More information about the security-dev mailing list