On Fri, 26 Jun 2026 08:39:17 GMT, Tobias Hartmann <[email protected]> wrote:

> Many of the new Valhalla compiler tests use the [IR 
> Framework](https://github.com/openjdk/jdk/blob/master/test/hotspot/jtreg/compiler/lib/ir_framework/README.md)
>  and should be run in `@run driver` mode because the framework will spawn a 
> new test VM and we want to avoid that the framework itself is affected by VM 
> flags. Unfortunately, this does not work with tests requiring 
> `--enable-preview` and we therefore need to run these tests in `@run main` 
> mode in Valhalla. `-Xcomp` and other expensive flags slow down execution 
> significantly, wasting machine hours and causing timeouts.
> 
> Jtreg 8.3 now finally supports driver mode with `--enable-preview` 
> ([CODETOOLS-7904153](https://bugs.openjdk.org/browse/CODETOOLS-7904153)), so 
> we can adjust the tests accordingly.
> 
> With these changes, run time of `TestLWorld#id0` with `-Xcomp`, for example, 
> improves by about **22.5%**.
> 
> I'll hold off with integration until the new jtreg version becomes the 
> default with [JDK-8386844](https://bugs.openjdk.org/browse/JDK-8386844).
> 
> Thanks,
> Tobias
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

This pull request has now been integrated.

Changeset: e07832f9
Author:    Tobias Hartmann <[email protected]>
URL:       
https://git.openjdk.org/valhalla/commit/e07832f9d9b9c2cb8e4a446c4fdaccfa48da266f
Stats:     195 lines in 26 files changed: 4 ins; 46 del; 145 mod

8381693: [lworld] IR Framework tests should run in jtreg driver mode

Reviewed-by: chagedorn

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

PR: https://git.openjdk.org/valhalla/pull/2585

Reply via email to