RFR: 8366908: Use a different class for testing JDK-8351654

Coleen Phillimore coleenp at openjdk.org
Fri Sep 5 15:33:11 UTC 2025


On Fri, 5 Sep 2025 12:53:49 GMT, Chen Liang <liach at openjdk.org> wrote:

>> java.time.Duration is a migrated class (to be a value type) in the Valhalla repo, so it's preloaded, which causes this test to fail.  Using a different java.base class achieves the purpose of this test.
>> Tested with tier1.
>
> test/hotspot/jtreg/runtime/verifier/CFLH/TestVerify.java line 167:
> 
>> 165:                 inst.addTransformer(new BadTransformer());
>> 166:                 Class<?> cls = Class.forName(CLASS_TO_BREAK);
>> 167:                 System.out.println("class loaded" + cls);
> 
> Isn't this println redundant, because the VerifyError is supposed to be thrown before? I think the old code uses `Durations.ofHours` to trigger verification and initialization of Duration.

It is not reached since we throw the VerifyError at line 166.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27116#discussion_r2325411714


More information about the hotspot-runtime-dev mailing list