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