[lworld] Integrated: 8263024: Convert Valhalla tests using the old framework to the new framework

Ekaterina Pavlova epavlova at openjdk.java.net
Thu Jul 15 16:08:50 UTC 2021


On Mon, 21 Jun 2021 21:38:43 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.
> 
> Testing:
>  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.
> 
> Thanks,
> Katya

This pull request has now been integrated.

Changeset: aa84c4ed
Author:    Ekaterina Pavlova <epavlova at openjdk.org>
Committer: David Simms <dsimms at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/aa84c4ed383b6266fbdb3fddd3aded1771613dfb
Stats:     5216 lines in 23 files changed: 1026 ins; 1275 del; 2915 mod

8263024: Convert Valhalla tests using the old framework to the new framework

Reviewed-by: thartmann, chagedorn

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

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



More information about the valhalla-dev mailing list