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

[Fei Yang]

On Mon, 19 Oct 2020 20:26:22 GMT, Vladimir Kozlov <k...@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