On Fri, 16 Aug 2024 05:41:01 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
> com/sun/jdi/ThreadMemoryLeakTest.java purposely runs with a small heap so it > is easier to detect a memory leak with an OOME. Running without compressed > oops uses more memory leading to an OOME even though there is no leak. We > should require that this test be run only with compressed oops. > > Tested by running just this test without compress oops and verifying that the > test is not run, and also running with compressed oops and verifying that it > is run. I also did an `-Xcomp` run to verify that it still is not run with > `-Xcomp`. This pull request has now been integrated. Changeset: d7281079 Author: Chris Plummer <cjplum...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/d72810794bf70f82e46f7220698e4d27d5973d5b Stats: 4 lines in 1 file changed: 1 ins; 0 del; 3 mod 8338482: com/sun/jdi/ThreadMemoryLeakTest.java requires that compressed oops are enabled Reviewed-by: amenkov, kevinw ------------- PR: https://git.openjdk.org/jdk/pull/20606
