RFR: 8366122: Shenandoah: Implement efficient support for object count after gc events [v2]

[Y. Srinivas Ramakrishna]

On Wed, 3 Sep 2025 01:31:24 GMT, pf0n <duke at openjdk.org> wrote:

>> src/hotspot/share/memory/heapInspection.hpp line 33:
>> 
>>> 31: #include "oops/objArrayOop.hpp"
>>> 32: #include "oops/oop.hpp"
>>> 33: #include "runtime/mutex.hpp"
>> 
>> Does this need to be in the header?
>
> I had an error in GHA during my early implementation, so I included that header. I'll remove and test to see if it is needed or not.

I believe you need it for this: https://github.com/pf0n/jdk/blob/f43e334bf76f878ec688cbd58a5ab1cb10deb0f9/src/hotspot/share/memory/heapInspection.hpp#L218

It probably happens to work at other places because of preceding includes that happen to satisfy this. Explicitly including it here makes sense for that reason. Not sure why it just started showing up, but likely a new use that didn't otherwise satisfy this include.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26977#discussion_r2320510893