RFR: 8292635: Improve CDSHeapVerifier testing [v2]
David Holmes
dholmes at openjdk.org
Wed Jan 18 02:21:25 UTC 2023
On Tue, 17 Jan 2023 17:40:33 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:
>> Moving the `ArchivedEnumTest.java` from `runtime/cds/appcds/cacheObject` to `jdk/internal/misc/CDS` so that the test will be run in jdk-tier1. This is to avoid inadvertently breakage of the test like in JDK-8292499.
>>
>> Testing: jdk-tier1, tier3, tier4.
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
>
> enable CDSHeapVerifier::verify() for non-debug build
I'm a bit confused. The JBS issue lists two things to do to improve verification, but all this PR seems to do is move the test - in which case the JBS issue should be updated.
This PR also changes the verification logic to be available in product builds - presumably because JDK testing uses product builds not debug ones - but this is not mentioned. Can I assume this change in itself has no impact unless verification is explicitly requested?
-------------
PR: https://git.openjdk.org/jdk/pull/11999
More information about the core-libs-dev
mailing list