Integrated: 8272856: DoubleFlagWithIntegerValue uses G1GC-only flag

Igor Ignatyev iignatyev at openjdk.java.net
Tue Aug 24 18:47:29 UTC 2021


On Mon, 23 Aug 2021 22:51:34 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

> Hi all,
> 
> could you please review the patch which replaces the flag used by `DoubleFlagWithIntegerValue` to `SweeperThreshold`? 
> 
> from JBS:
>> `runtime/CommandLine/DoubleFlagWithIntegerValue.java` test uses `G1ConcMarkStepDurationMillis` flag which is available only on the builds w/ G1 GC, hence this test might fail on the builds where G1 is disabled. 
>> instead of adding `@requires` to exclude the test from running on builds w/o G1, the test can be updated to use the flag that is available in all builds.
> 
> testing: `runtime/CommandLine/DoubleFlagWithIntegerValue.java`  on `{linux,windows,macosx}-x64`
> 
> Thanks,
> -- Igor

This pull request has now been integrated.

Changeset: 6ace805f
Author:    Igor Ignatyev <iignatyev at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/6ace805f8c69b273bf998f084f9651bc5bcd52b6
Stats:     2 lines in 1 file changed: 0 ins; 0 del; 2 mod

8272856: DoubleFlagWithIntegerValue uses G1GC-only flag

Reviewed-by: dholmes, kbarrett

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

PR: https://git.openjdk.java.net/jdk/pull/5233


More information about the hotspot-runtime-dev mailing list