On Fri, 15 May 2026 11:16:59 GMT, Coleen Phillimore <[email protected]> wrote:
>> 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 49 commits: >> >> - Merge branch 'master' into remove-jvmci >> - Remove graal from test/langtools >> - Remove jvmci and graal from test/lib >> - Remove jvmci and graal from test/jdk/jdk/jfr >> - Remove jvmci from test/jdk/tools/jlink >> - Remove jvmci and graal from test/jdk/jdk/modules >> - Remove jvmci and graal frmo test/jdk/java/lang >> - Remove graal from test/jdk/javax >> - Remove jvmci and graal from test/jdk/com/sun/jdi >> - Remove jvmci and graal from test/hotspot/jtreg/runtime >> - ... and 39 more: https://git.openjdk.org/jdk/compare/061bd076...0100264f > > src/hotspot/share/classfile/javaClasses.hpp line 1767: > >> 1765: }; >> 1766: >> 1767: class java_lang_Integer : AllStatic { > > Can you remove these classes too? I think the references are only the > Symbol* for the names. I could be wrong. I don't think we should do it. IT is not related to JVMCI. And C2 may use it for "boxing" optimizations. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30834#discussion_r3248880372
