[lworld] Integrated: 8350632: [lworld] Parse::array_store_check() assert introduced by 8348411 fails

Christian Hagedorn chagedorn at openjdk.org
Wed Mar 26 14:10:22 UTC 2025


On Wed, 26 Mar 2025 11:37:13 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

> The assert introduced with JDK-8348411 is too strong when having value classes around because of the following reason:
> 
> https://github.com/openjdk/valhalla/blob/3f6f249bd457538f020d062ffe5b295509b45a4c/src/hotspot/share/opto/parseHelper.cpp#L265-L267
> 
> I've updated the assert to make it work with value classes by special casing them.
> 
> Thanks,
> Christian

This pull request has now been integrated.

Changeset: dd0b9589
Author:    Christian Hagedorn <chagedorn at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/dd0b9589f5db0784ffe102e8952e742ea60e0852
Stats:     10 lines in 1 file changed: 8 ins; 2 del; 0 mod

8350632: [lworld] Parse::array_store_check() assert introduced by 8348411 fails

Reviewed-by: thartmann

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

PR: https://git.openjdk.org/valhalla/pull/1410


More information about the valhalla-dev mailing list