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

Fei Yang fyang at openjdk.java.net
Sat Oct 10 13:09:11 UTC 2020


On Fri, 9 Oct 2020 17:35:22 GMT, Andrew Haley <aph at openjdk.org> wrote:

> I see Linux x64 failed. However, I don't seem to be able to withdraw my patch approval.
> However, please consider it withdrawn.

Thanks for approving this patch.
I checked the error messages and I think the failures were not caused by this patch.
The failures has been fixed by the following two commits:
    commit ec41046c5ce7077eebf4a3c265f79c7fba33d916
    8254348: Build fails when cds is disabled after JDK-8247536
    commit aaa0a2a04792d7c84150e9d972790978ffcc6890
    8254297: Zero and Minimal VMs are broken with undeclared identifier 'DerivedPointerTable' after JDK-8253180

The testing was triggered again automatically after I merge master and I see it passed now.
Do you have any comments for the discussion here?
    https://github.com/openjdk/jdk/pull/207#issuecomment-701243662

Valerie Peng has checked the java security changes, i.e. src/java.base/share/classes/sun/security/provider/SHA3.java.
Do you think we need another reviewer for this patch?

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

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



More information about the security-dev mailing list