On Fri, 9 Jan 2026 17:22:04 GMT, Larry Cable <[email protected]> wrote:
>> src/hotspot/share/oops/instanceKlass.cpp line 2360: >> >>> 2358: >>> 2359: InstanceKlass *ik = k->is_instance_klass() ? InstanceKlass::cast(k) >>> : nullptr; >>> 2360: >> >> Looks good with the new local InstanceKlass ik. >> Is it possible to not reassign into ik at lines 2386 and 2397, i.e. ik was >> the target class as an instanceKlass, but later represents pd or cs, so that >> can be hard to follow. > > I view "ik" like "i" in a for loop, I dont think that defining two additional > InstanceKlass'es for "pd" or "cs" when their values are subsequently only > used once is merited, but its subjective. ok no problem ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/29048#discussion_r2677101240
