On Thu, 16 Sep 2021 17:53:25 GMT, Mandy Chung <[email protected]> wrote:
> > The fix in the test case seems fine to me.
>
> This test may fail in other cases. We should prepare for those cases in this
> fix:
>
> ```
> if (!Platform.isDefaultCDSArchiveSupported() && !Platform.isAArch64()))
> throw new SkippedException("not a supported platform");
> ```
I just noticed there was a bug fix
[JDK-8269840](https://bugs.openjdk.java.net/browse/JDK-8269840) (Update
Platform.isDefaultCDSArchiveSupported() to return true for aarch64 platforms).
So the second condition in the above is not necessary.
>
> Calvin suggests to throw SkippedException in an offline chat.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5519
