RFR: 8268389: [lworld] Tests must be update after core reflection changes for the L/Q model

Harold Seigel hseigel at openjdk.java.net
Tue Jun 8 13:48:31 UTC 2021


On Tue, 8 Jun 2021 13:17:20 GMT, Frederic Parain <fparain at openjdk.org> wrote:

> Fix all but one test to pass with the new L/Q model and the new core reflection.
> The failing test, TestIntrinsics.java might be a real VM bug and not a test bug, but it is C2 related.
> 
> Fred

test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestIntrinsics.java line 75:

> 73:         Asserts.assertTrue(test1(MyValue1.class, MyValue1.class), "test1_2 failed");
> 74:         Asserts.assertTrue(test1(MyValue1.class.asValueType(), MyValue1.class.asValueType()), "test1_3 failed");
> 75:         Asserts.assertTrue(test1(MyValue1.class.asValueType(), MyValue1.class.asValueType()), "test1_4 failed");

should the first MyValue1.class.asValueType() in line 75 be MyValue1.class (to match MyValue1.ref.class) ?

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

PR: https://git.openjdk.java.net/valhalla/pull/437



More information about the valhalla-dev mailing list