On Tue, 24 Mar 2026 13:20:59 GMT, Coleen Phillimore <[email protected]> wrote:
>> Frederic Parain has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Apply suggestion from review > > src/hotspot/share/classfile/fieldLayoutBuilder.cpp line 1385: > >> 1383: Array<int>* super_map = ik->acmp_maps_array(); >> 1384: assert(super_map != nullptr, "super class must have an acmp map"); >> 1385: int nb_nonoop_field = super_map->at(0); > > Is 'nb' next block? No, number. In the acmp_map format described in classFileParser.cpp, the first element of the array is `number_of_nonoop_entries`. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2248#discussion_r2981612147
