[lworld] Integrated: 8324117: [lworld] C1 hits "Can not patch access to flat field" assert after JDK-8320437

Tobias Hartmann thartmann at openjdk.org
Tue Feb 6 17:40:12 UTC 2024


On Tue, 6 Feb 2024 16:08:34 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> Current code assumes that even if the holder is not loaded, we always know at compile time if a field type is an inline type. That's not guaranteed after the removal of Q-types and as a result we hit an assert when trying to patch a flat field at runtime.
> 
> I don't remember why we had this complicated code before but we can simply deopt at runtime when we detect the flat field during patching. The code will then be re-compiled.
> 
> Thanks,
> Tobias

This pull request has now been integrated.

Changeset: 02b59630
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/02b5963020e5f81db85d231a911672960bf72c00
Stats:     48 lines in 3 files changed: 9 ins; 38 del; 1 mod

8324117: [lworld] C1 hits "Can not patch access to flat field" assert after JDK-8320437

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

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



More information about the valhalla-dev mailing list