[lworld] Integrated: 8375196: [lworld] TestArrays triggers assert in LRG::set_num_regs

Quan Anh Mai qamai at openjdk.org
Fri Jan 16 02:45:37 UTC 2026


On Wed, 14 Jan 2026 23:38:13 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:

> Hi,
> 
> The issue is that `PhaseMacroExpand::value_from_mem_phi` does not take into consideration flat arrays, so it incorrectly returns the whole object for a load from one of its fields. This results in a byte `Phi` with an input being an `oop`, triggering the assert during `LRG::set_num_regs` because the size of a `Phi` and its input are different.
> 
> Please take a look and leave your reviews, thanks a lot.

This pull request has now been integrated.

Changeset: c2b08c8c
Author:    Quan Anh Mai <qamai at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/c2b08c8ca133afe5dfef391519196760a1840807
Stats:     100 lines in 3 files changed: 61 ins; 29 del; 10 mod

8375196: [lworld] TestArrays triggers assert in LRG::set_num_regs

Reviewed-by: thartmann

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

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


More information about the valhalla-dev mailing list