RFR: 8272856: DoubleFlagWithIntegerValue uses G1GC-only flag
Kim Barrett
kbarrett at openjdk.java.net
Tue Aug 24 08:10:24 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
Looks good.
-------------
Marked as reviewed by kbarrett (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/5233
More information about the hotspot-runtime-dev
mailing list