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