On Fri, 15 May 2026 15:39:26 GMT, Chris Plummer <[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/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/runtime/CompilerThread.java
>  line 54:
> 
>> 52:   public boolean isHiddenFromExternalView() {
>> 53:       // CompilerThreads are always hidden.
>> 54:       return true;
> 
> With this change we actually don't need the CompilerThread class anymore as 
> this is the only override of the JavaThread class. A small tweek is need in 
> Threads.java if it is removed. I can do that as a separate issue if you'd 
> like. Also, [JDK-8348317](https://bugs.openjdk.org/browse/JDK-8348317) can be 
> closed.

I gave it a shot with 640de4f2268.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30834#discussion_r3258524429

Reply via email to