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

Fei Yang fyang at openjdk.java.net
Tue Oct 20 08:07:24 UTC 2020


On Mon, 19 Oct 2020 20:26:22 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

> Always run graalunit testing with new intrinsics. You need to adjust Graal test:
> src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.hotspot.test/src/org/graalvm/compiler/hotspot/test/CheckGraalIntrinsics.java

Thanks for looking at this. We did run graalunit testing and added the following change in our first commit:

diff --git
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
index f0e17947460..8f3f4ed9323 100644
--- 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
@@ -601,6 +601,7 @@ public class CheckGraalIntrinsics extends GraalTest {
          if (!config.useSHA512Intrinsics()) {
              add(ignore, "sun/security/provider/SHA5." + shaCompressName + "([BI)V");
          }
+        add(toBeInvestigated, "sun/security/provider/SHA3." + shaCompressName + "([BI)V");
      }

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

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



More information about the security-dev mailing list