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

Doug Simon dnsimon at openjdk.org
Fri Apr 18 15:35:41 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 refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:

  impose a length limit on InstalledCode.name

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

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

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

  Stats: 3 lines in 1 file changed: 2 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