RFR: 8356783: CompilerTask hot_method is redundant

Aleksey Shipilev shade at openjdk.org
Tue May 13 08:33:52 UTC 2025


On Mon, 12 May 2025 14:02:43 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> This gave me some grief when implementing [JDK-8231269](https://bugs.openjdk.org/browse/JDK-8231269). From the initializations, it looks to me that `CompilerTask::hot_method()` is either `method()` or `nullptr`. In both cases, we do nothing special. So tracking `hot_method` is redundant, and can be purged. This improves performance a little, since it avoids extra handle-izing across compiler code, and of course it simplifies coding as well.
> 
> Additional testing:
>  - [x] Linux x86_64 server fastdebug, `compiler/`
>  - [x]  Linux x86_64 server fastdebug, `all`

Thanks! Testing is green here. I'll wait a bit more if anyone else wants to review, and then I'll integrate to continue with [JDK-8231269](https://bugs.openjdk.org/browse/JDK-8231269).

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

PR Comment: https://git.openjdk.org/jdk/pull/25185#issuecomment-2875537533


More information about the graal-dev mailing list