On Mon, 30 Oct 2023 12:13:59 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? We would have to run across all of the mach5 platforms to verify. You could remove it for now and we go from here ------------- PR Comment: https://git.openjdk.org/jdk/pull/12991#issuecomment-1785341362
