[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