[lworld] RFR: 8371357: [lworld] Remove EnableValhalla [v2]

Paul Hübner phubner at openjdk.org
Mon Dec 15 16:45:53 UTC 2025


On Mon, 15 Dec 2025 14:50:21 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> Paul Hübner has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Remove wrong comment.
>>  - Fix imports.
>
> Looks good minus one nit.

Thanks for the feedback, everyone. After a face-to-face discussion, we decided this shape (without explicit feature granularity or its own file) is sufficient for our current needs. Sole `--enable-preview` for Valhalla is a large change that will be delivered incrementally. This is only the start, and many subsequent changes depend on this change.  I've filed a follow-up RFE https://bugs.openjdk.org/browse/JDK-8373694, where we can devote more (non-blocking) time and discussion towards how a better way of checking may look. Ideally, @stefank's mainline changes will have made it into lworld by then too.

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

PR Comment: https://git.openjdk.org/valhalla/pull/1759#issuecomment-3656622888


More information about the valhalla-dev mailing list