RFR: 8267265: Use new IR Test Framework to create tests for C2 IGV transformations

Igor Ignatyev iignatyev at openjdk.java.net
Tue Aug 17 17:02:26 UTC 2021


On Tue, 17 Aug 2021 16:24:37 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

> This directory was created to split original set of tests to run in parallel and not related to C2 compiler

aren't you mixing this up w/ `tier1_compiler_[1-3]` test group? if you aren't, this was the wrong way to split tests and we need to fix that.

>  I want to have directories in `/compiler` per `optimization/transformation` type as we have now. For example, `/loopopts` is for C2 only but it is not under `/c2`.
> Actually I want to clear `c2` and `c1` directories eventually to move tests to corresponding category directories instead.

I would prefer to clear `/compiler` directory and move c1-specific tests into `/compiler/c1` directory and c2-specific tests into `/compiler/c2` with optimization/transformation type being subdirectories inside `/compiler/c[12]` (and shared/common tests moved to `/compiler/shared`), so `loopopts` tests are to moved to `/compiler/c2/loopopts`. one of the reasons, most of optimizations tests will be compiler-specific and having them together in `/compiler` doesn't make much sense and just makes it harder to navigate/use test base. e.g. if we have all c2 tests in `/compiler/c2`, one can easily run them together to test c2 changes, if c2-specific tests are in several directories in `/compiler`, so one will have to have intrinsic knowledge about all tests in `/compiler` to select the right one.

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

PR: https://git.openjdk.java.net/jdk/pull/5135


More information about the hotspot-dev mailing list