On Tue, 18 Aug 2026 07:25:33 GMT, Markus Grönlund <[email protected]> wrote:
>> src/hotspot/share/classfile/vmClasses.cpp line 274: >> >>> 272: Dictionary* dictionary = loader_data->dictionary(); >>> 273: dictionary->add_klass(THREAD, klass->name(), klass); >>> 274: >> >> Why the reordering of the adding to the dictionary here? > > To align with all other definition orderings: > > 1. Add to hierarchy. > 2. Add to dictionary. > > No reason for why this site would have it the other way. Have you confirmed that? It may be the order doesn't matter, but it seems an unnecessary change to make unless you want to verify that. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32218#discussion_r3802078558
