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

Tobias Hartmann thartmann at openjdk.org
Thu Oct 20 14:11:29 UTC 2022


On Thu, 20 Oct 2022 12:21:05 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: 4e59de18
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/4e59de18738484a0258f45cd6a49b029f352b3a2
Stats:     4 lines in 2 files changed: 0 ins; 0 del; 4 mod

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

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

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



More information about the valhalla-dev mailing list