[lworld] RFR: 8295731: [lworld] C1's linear register allocator spill map is incorrect

Tobias Hartmann thartmann at openjdk.org
Thu Oct 20 12:28:11 UTC 2022


We spuriously assert on AArch64 because the spill location on stack for an 8-byte value is not properly aligned. After some hours of debugging and staring at the Valhalla specific changes, I finally noticed that when moving around the `T_VALUE`/`T_INLINE_TYPE`/`T_PRIMITIVE_OBJECT` basic type definitions, we forgot to adjust the spill map of C1's register allocator accordingly.

Thanks,
Tobias

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

Commit messages:
 - 8295731: [lworld] C1's linear register allocator spill map is incorrect

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

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



More information about the valhalla-dev mailing list