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

Christian Hagedorn chagedorn at openjdk.org
Wed Mar 26 11:42:55 UTC 2025


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

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

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

Changes: https://git.openjdk.org/valhalla/pull/1410/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1410&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8350632
  Stats: 10 lines in 1 file changed: 8 ins; 2 del; 0 mod
  Patch: https://git.openjdk.org/valhalla/pull/1410.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1410/head:pull/1410

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


More information about the valhalla-dev mailing list