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

[Yudi Zheng]

On Sat, 19 Apr 2025 20:16:24 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.
>
> Doug Simon has updated the pull request incrementally with one additional commit since the last revision:
> 
>   InstalledCode.name can be null

LGTM!

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

Marked as reviewed by yzheng (Committer).

PR Review: https://git.openjdk.org/jdk/pull/24753#pullrequestreview-2780278854