On Tue, 21 Apr 2026 08:05:02 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 contains 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). test/jdk/java/lang/Class/getDeclaredField/FieldSetAccessibleTest.java line 280: > 278: moduleToDeps.put(md.name(), deps); > 279: inverseDeps.put(md.name(), new HashSet<>()); > 280: } Since this no longer filters the jdk.graal.* modules then it should be possible to remove moduleToDeps and inverseDeps too, and simplify it down to return ModuleFinder.ofSystem() .findAll() .stream() .map(mref -> mref.descriptor().name()) .collect(Collectors.toSet()); ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30834#discussion_r3246907831
