Integrated: 8356783: CompilerTask hot_method is redundant
Aleksey Shipilev
shade at openjdk.org
Tue May 13 13:20:07 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`
This pull request has now been integrated.
Changeset: 48d2acb3
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/48d2acb3860f742eb1c06b89f8a7208d0d7a01e7
Stats: 62 lines in 8 files changed: 0 ins; 47 del; 15 mod
8356783: CompilerTask hot_method is redundant
Reviewed-by: kvn, cslucas
-------------
PR: https://git.openjdk.org/jdk/pull/25185
More information about the graal-dev
mailing list