On Tue, 19 May 2026 07:54:00 GMT, Manuel Hässig <[email protected]> wrote:
>> This PR removes the experimental JVMCI feature and all usages and references >> of it, including references to Graal. For more details on the rationale, >> please see the issue description. >> >> This change was constructed in several steps: >> 1. Remove all folders directly related to JVMCI. >> 2. Go through one folder at a time and remove code relating to matches for >> the regex `([Jj][Vv][Mm][Cc][Ii])|([Gg]raal)|vm.ci|[Gg]alahad` and >> immediately obvious dead code. >> 3. Filter the `git blame` for each file for JVMCI related commits and check >> if that code is now dead. >> >> ### Note to Reviewers >> >> Each commit is limited to a subfolder or a change spanning further so you >> can take look at a piece of the codebase you are familiar with. I recommend >> you look at individual commits and select "Ignore Whitespace" in the Github >> UI. >> >> ### Testing >> >> - [x] Github Actions >> - [x] tier1-6 linux-x64, linux-aarch64, windows-x64, macosx-aarch64, >> macosx-x64 >> - [x] build linux-zero, linux-arm, linux-riscv, linux-ppc (LE & BE) >> fastdebug with features C1+C2, C2 only, C1 only >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Manuel Hässig has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains 59 commits: > > - Merge branch 'master' into remove-jvmci > - Copyright years > - Merge branch 'master' into remove-jvmci > - serviceablity: remove CompilerTread.java > - Remove unused boxed primitive classes > - More dead code > > Co-Authored-By: Vladimir Ivanov <[email protected]> > - Fix some asserts > > Co-Authored-By: Vladimir Kozlov <[email protected]> > - Simplify ClassNameJrtStreamBuilder::systemModules > > Co-Authored-By: Alan Bateman <[email protected]> > - Add back JVMCI flags to description of -XX:+NeverActAsServerClassMachine > - Merge branch 'master' into remove-jvmci > - ... and 49 more: https://git.openjdk.org/jdk/compare/45e66b6a...3f822c8f >> I don't know if we want to remove this file or do the jdk.jsobject-style >> removal in the symbols - it might be okay for us to nuke these internal >> modules this way. > I think removing the files for JDK releases from "M" (i.e. 22) through "R" > (i.e. 27) i needs some more consideration, Removing the files in the current > release is much less problematic. Note, JVMCI was always JDK internal experimental feature. `javac --release` applies to **`public Java API`** I am not sure what the goal of CSR will be here. ------------- PR Comment: https://git.openjdk.org/jdk/pull/30834#issuecomment-4490113750
