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