RFR: 8324514: ClassLoaderData::print_on should print address of class loader [v3]

Aleksey Shipilev shade at openjdk.org
Tue Jan 23 15:22:41 UTC 2024


On Tue, 23 Jan 2024 14:50:35 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> If not peek(), I kind of think I'd prefer the !_unloading test more than a peek_raw().
>
> I haven't looked at this change at all, and I can already say I completely agree with @coleenp here.

Well, yes, it is dangerous in the sense that you cannot dereference the `oop` safely after this. It is only useful for debugging printouts. We can do `!unloading` too, like the very first commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17534#discussion_r1463441478


More information about the hotspot-runtime-dev mailing list