[lworld] RFR: 8366701: [lworld] Method profiles for AOT cache introduced with JEP 515 miss Valhalla specific profiling

Tobias Hartmann thartmann at openjdk.org
Thu Sep 11 07:07:31 UTC 2025


On Thu, 11 Sep 2025 06:58:59 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

> JEP 515 extended the AOT cache to also collect method profiles during training runs. In Valhalla, we have additional profiling for:
> - Array store
> - Array load
> - Acmp
> 
> When JEP 515 was merged in, we did to account for these Valhalla specific profiling information and later crash when trying to create the AOT cache. The fix is straight forward to add the missing collection code for the Valhalla specific profiling.
> 
> I also unproblemlisted the affected tests.
> 
> Testing:
> - tier1-4 + stress
> 
> Thanks,
> Christian

Looks good and trivial to me!

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

Marked as reviewed by thartmann (Committer).

PR Review: https://git.openjdk.org/valhalla/pull/1568#pullrequestreview-3209317249


More information about the valhalla-dev mailing list