On Wed, 14 Jan 2026 23:38:13 GMT, Quan Anh Mai <[email protected]> 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 <[email protected]> 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
