On Sun, 12 Mar 2023 11:21:18 GMT, Eirik Bjorsnos <d...@openjdk.org> wrote:
> I think the changes look good overall. Thank you for this. I am not sure that > the `@requires` is needed at this point. Was the `@requires (sun.arch.data.model == "64")` added to satisfy the 8GB memory requirement? If so, I guess we can safely remove the requires tag, even though the test still creates a ~2GB sparse file? ------------- PR Comment: https://git.openjdk.org/jdk/pull/12991#issuecomment-1785057458
