[lworld] RFR: 8373987: [lworld] exploded-image/test broken since disable patching (JDK-8373806)

Roger Riggs rriggs at openjdk.org
Mon Dec 22 15:00:57 UTC 2025


On Sat, 20 Dec 2025 14:54:27 GMT, David Beaumont <duke at openjdk.org> wrote:

> Preview mode support for exploded build.
> 
> This is an alternate approach from https://github.com/openjdk/valhalla/pull/1845 which I think is actually cleaner and more robust.

src/hotspot/share/classfile/classLoader.cpp line 276:

> 274: 
> 275: // Returns the access mode for an initialized JImage file (reflects --enable-preview).
> 276: static bool is_preview_enabled() {

Previously, the preview related names were associated with jimage mode, expanding that to the exploded image raises a question about the name and the name set_preview_mode ; there may be some ambiguity in this is_preview_enabled() and the vm wide method(s) indicating --enable-preview across all HotSpot functions.
It would be good if this name could reflect the mode of loading classes.
Maybe...
is_class_loading_mode_preview() based on class_loading_mode.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1854#discussion_r2640102161


More information about the valhalla-dev mailing list