[lworld] RFR: 8375196: [lworld] TestArrays triggers assert in LRG::set_num_regs [v2]
Tobias Hartmann
thartmann at openjdk.org
Thu Jan 15 08:31:35 UTC 2026
On Thu, 15 Jan 2026 07:06:34 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.
>
> Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:
>
> Add reminder at LibraryCallKit::inline_newArray
Thanks!
-------------
Marked as reviewed by thartmann (Committer).
PR Review: https://git.openjdk.org/valhalla/pull/1909#pullrequestreview-3664430347
More information about the valhalla-dev
mailing list