RFR: 8268095: CDS MethodHandle tests should add -XX:-VerifyDependencies

Calvin Cheung ccheung at openjdk.java.net
Thu Jun 3 18:03:33 UTC 2021


On Thu, 3 Jun 2021 16:49:23 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Please review this patch for disabling the `VerifyDependencies` option when running the CDS MethodHandle tests with debug build. It reduces the time required for running the tests a lot.
>> e.g. total time for running the `appcds/methodHandles/MethodHandlesSpreadArgumentsTest.java` test with debug build:
>> before 489.102s
>> after 62.721s
>> 
>> The patch passed tier1 and tier2 tests.
>
> Looks good. As I can see, the changes are in the .sh files, and the .java files are mechanically regenerated from the .sh files.

@iklam Thanks for the review.

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

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


More information about the hotspot-runtime-dev mailing list