Integrated: 8355034: [JVMCI] assert(static_cast<int>(_jvmci_data_size) == align_up(compiler->is_jvmci() ? jvmci_data->size() : 0, oopSize)) failed: failed: 104 != 16777320

[Doug Simon]

On Fri, 18 Apr 2025 13:27:02 GMT, Doug Simon <dnsimon at openjdk.org> wrote:

> After [JDK-8343789](https://bugs.openjdk.org/browse/JDK-8343789), the size of a `JVMCINMethodData` object is limited to `uint16_t`. This object embeds the value of `InstalledCode.name` so effectively imposes a limit on the name length. This PR establishes an upper limit on the name value as this name should only be for informative purposes when inspecting compiled code.
> 
> While debugging the problem that exposed the limit, it was confusing that `-XX:+PrintCompilation` did not show the name so this PR builds on [JDK-8336760](https://bugs.openjdk.org/browse/JDK-8336760) to add the name in `PrintCompilation` output for JVMCI "hosted" methods.

This pull request has now been integrated.

Changeset: 2f7806ff
Author:    Doug Simon <dnsimon at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/2f7806ffe5b5b4b2f7caa14d4559943968c34678
Stats:     83 lines in 4 files changed: 80 ins; 0 del; 3 mod

8355034: [JVMCI] assert(static_cast<int>(_jvmci_data_size) == align_up(compiler->is_jvmci() ? jvmci_data->size() : 0, oopSize)) failed: failed: 104 != 16777320

Reviewed-by: never, yzheng, cslucas

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

PR: https://git.openjdk.org/jdk/pull/24753