RFR: 8340733: Add scope for relaxing constraint on JavaCalls from CompilerThread [v5]

Tomáš Zezula duke at openjdk.org
Thu Oct 31 12:38:17 UTC 2024


On Thu, 10 Oct 2024 07:29:46 GMT, Doug Simon <dnsimon at openjdk.org> wrote:

>> Tomáš Zezula has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Simplified C2V_BLOCK.
>
> src/hotspot/share/compiler/compilerThread.cpp line 58:
> 
>> 56: 
>> 57: void CompilerThread::set_compiler(AbstractCompiler* c) {
>> 58:   /*
> 
> The comment could be a little shorter:
> 
>   /*
>    * Compiler threads need to make Java upcalls to the jargraal compiler.
>    * Java upcalls are also needed by the InterpreterRuntime when using jargraal.
>    */

Resolved in https://github.com/openjdk/jdk/pull/21285/commits/7e0f1a4227f388dc8e22e6200dc026f056d26eed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21285#discussion_r1824373085


More information about the hotspot-compiler-dev mailing list