[lworld] RFR: 8376221: [lworld] Do not store array of InlineLayoutInfo for all InstanceKlasses [v4]
Joel Sikström
jsikstro at openjdk.org
Thu Jan 29 09:26:24 UTC 2026
On Thu, 29 Jan 2026 08:38:43 GMT, Paul Hübner <phubner at openjdk.org> wrote:
>> The logic is the same as we had before, I just moved it to this helper. With that said, I'm not sure why we check this, as an injected field should likely not be present in `_inline_layout_info_array` so should not get through this check at all...
>
> In that case, can we convert the injected check to an assertion? And perhaps see what happens?
I did some local testing and we need the injected check there since the injected field is not accounted for in the count of how many fields there are in the klass, so it does not have an entry in the `_inline_layout_info_array`.
We could have a check like this, but I'd rather keep only having checks that return true, and returning false in the end of the function.
if (fieldinfo.field_flags().is_injected()) {
return false;
}
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1966#discussion_r2740698455
More information about the valhalla-dev
mailing list