[lworld] RFR: 8263024: Convert Valhalla tests using the old framework to the new framework [v2]
chagedorn at openjdk.java.net
Tue Jul 13 13:03:13 UTC 2021
On Tue, 13 Jul 2021 06:24:39 GMT, Ekaterina Pavlova <epavlova at openjdk.org> wrote:
>> This PR converts compiler/valhalla/inlinetypes tests based on old Valhalla/IR test framework to new one
>> which was recently integrated by Christian as part of JDK-8254129.
>> The tests which were we decided to convert are basically the ones which extends InlineTypeTest class.
>> The rest of compiler/valhalla/inlinetypes tests were agreed (with Christian and Tobias) to keep untouched
>> as they are in most cases regression tests.
>> The conversion rules/approach is pretty well described in the bug's description section (see JDK-8263024).
>> A detailed description of new IR test framework could found in test/hotspot/jtreg/compiler/lib/ir_framework/README.md.
>> Ran compiler/valhalla/inlinetypes in all the configurations used in hs-tier1-9.
>> There 2 failed converted tests:
>> compiler/valhalla/inlinetypes//TestNullableArrays.java - new bug JDK-8269070 filed
>> compiler/valhalla/inlinetypes/TestIntrinsics.java - know issue tracked by JDK-8239003
>> Big thanks to Christian and Tobias for advice and help during this tests conversion.
>> Please review the changes.
> Ekaterina Pavlova has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
> - Merge lworld
> - Converted to new IR framework
test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestLWorld.java line 3865:
> 3863: int result = test141();
> 3864: Asserts.assertEquals(result, 0);
> 3865: }
I've just seen that there is one more test below (`test142`) which was not converted, yet.
More information about the valhalla-dev