On Fri, 6 Oct 2023 20:56:13 GMT, Hannes Greule <hgre...@openjdk.org> wrote:
> See the bug description for more information.
>
> This implementation brings down the time to take a heap dump on the example
> application in the bug report to <2 seconds on my machine.
src/hotspot/share/oops/fieldStreams.hpp line 194:
> 192: void prepare() {
> 193: _next_klass = next_klass_with_fields();
> 194: // special case: the base klass has no fields. If any supertype has
> any fields, use that directly.
"base klass" sounds misleading.
I think "initial" would be clearer
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16083#discussion_r1353752601
