[lworld] RFR: 8373622: [lworld] Enabling jimage changes break tests with inMemoryJavaCompiler
Chen Liang
liach at openjdk.org
Mon Dec 15 15:56:18 UTC 2025
On Fri, 12 Dec 2025 20:53:03 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
> This change tests the properties: java.test.opts and the property for test.enable.preview (set when test has @enablePreview), and will pass --enable-preview -source 26 to the inMemoryJavaCompiler compiler task. I don't really like this but the alternative is to do the same thing from all the tests, and pass this down as parameters. Better alternative ideas are welcome.
>
> When the jimage code is checked in, a bunch of tests fail because of this problem.
>
> Testing with tier1-4.
test/lib/jdk/test/lib/compiler/InMemoryJavaCompiler.java line 275:
> 273: opts.add("26");
> 274: }
> 275: }
if (System.getProperty("test.java.opts", "").contains("--enable-preview")
|| System.getProperty("test.enable.preview", "").equals("true")) {
opts.add("--enable-preview");
opts.add("-source");
opts.add(Integer.toString(Runtime.version().feature()));
}
I recommend using `Runtime.version().feature()` instead of hardcoded version.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1797#discussion_r2619970670
More information about the valhalla-dev
mailing list