On Wed, 21 Oct 2020 19:20:28 GMT, Vladimir Kozlov <[email protected]> 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
