On Thu, 15 Jan 2026 07:06:34 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.
>
> Quan Anh Mai has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Add reminder at LibraryCallKit::inline_newArray

Thanks very much for your review!

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

PR Comment: https://git.openjdk.org/valhalla/pull/1909#issuecomment-3757844929

Reply via email to