[jdk17u-dev] RFR: 8332252: Clean up vmTestbase/vm/share

[Martin Doerr]

On Mon, 16 Jun 2025 09:22:19 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

> I backport this for parity with 17.0.17-oracle.
> 
> I had to touch three files:
> 
> test/hotspot/jtreg/vmTestbase/metaspace/gc/HighWaterMarkTest.java
> Resolved Copyright because "8284853: Fix various 'expected' typo" is not in 17.
> 
> 
> test/hotspot/jtreg/vmTestbase/metaspace/staticReferences/StaticReferences.java
> Resolved imports because later change "8202100: Merge vm/share/InMemoryJavaCompiler w/ jdk/test/lib/compiler/InMemoryJavaCompiler" was already backported.
> 
> test/hotspot/jtreg/vmTestbase/vm/share/ProcessUtils.cpp
> Several changes are missing here. These are all cleanups that touch code that is removed anyways.
> Resolved.
> 
> I ran the modified tests. Where a aux file has been modified, I selected some test that uses the coding as assured it is passing.

test/hotspot/jtreg/vmTestbase/metaspace/gc/watermark_0_1/TestDescription.java line 35:

> 33:  * @requires vm.gc != null | !vm.opt.final.ClassUnloadingWithConcurrentMark
> 34:  * @requires vm.gc != "G1" | !vm.opt.final.ClassUnloadingWithConcurrentMark
> 35:  * @requires vm.gc != "Z"

Can you explain this change, please? I can't see it in the original change.

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

PR Review Comment: https://git.openjdk.org/jdk17u-dev/pull/3640#discussion_r2151683566