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]

Doug Simon dnsimon at openjdk.org
Sat Apr 19 20:16:24 UTC 2025


> After [JDK-8343789](https://bugs.openjdk.org/browse/JDK-8343789), the size of a `JVMCINMethodData` object is limited to `uint16_t`. That object embeds the value of `InstalledCode.name` so effectively imposes a limit on the name length. This PR establishes a lower 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

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/24753/files
  - new: https://git.openjdk.org/jdk/pull/24753/files/be003f35..f7c01793

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=24753&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=24753&range=01-02

  Stats: 6 lines in 2 files changed: 5 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/24753.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24753/head:pull/24753

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


More information about the graal-dev mailing list