RFR: 8354145: G1: UseCompressedOops boundary is calculated on maximum heap region size instead of maxiumum ergonomic heap region size [v2]

[Thomas Schatzl]

On Tue, 15 Apr 2025 02:49:52 GMT, Tongbao Zhang <duke at openjdk.org> wrote:

>> test/hotspot/jtreg/gc/arguments/TestG1CompressedOops.java line 35:
>> 
>>> 33:  * @modules java.management/sun.management
>>> 34:  * @library /test/lib
>>> 35:  * @library /
>> 
>> Why this line is needed? I don't see any dependencies on "/"
>> If you use some test code outside directory, better to build them.
>
>> Why this line is needed? I don't see any dependencies on "/" If you use some test code outside directory, better to build them.
> 
> Yes, the GCArguments depends on the  ```@library /``` , many tests in ``` test/hotspot/jtreg/gc/arguments``` use this

Afaict `GCArguments` only depends on `test.lib.*` too.
Other than that, the use of the `@library`directives is often just copy&paste without particular meaning.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24541#discussion_r2065747094