[lworld] RFR: 8336779: [lworld] C2 compilations hits "what's left behind is null" assert in do_checkcast

Tobias Hartmann thartmann at openjdk.org
Mon Aug 12 14:32:30 UTC 2024


We assert because we don't expect an `InlineTypeNode` after a `null_assert`. The fix is to simply replace it with null in the parsing map. Full fix to enable scalarization of null after `null_assert` will be done by [JDK-8284443](https://bugs.openjdk.org/browse/JDK-8284443) (as the ToDo in the code at line 2198 already mentions).

Best regards,
Tobias

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

Commit messages:
 - Removed ToDo
 - 8336779: [lworld] C2 compilations hits "what's left behind is null" assert in do_checkcast

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

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


More information about the valhalla-dev mailing list