On Tue, 19 Sep 2023 22:48:16 GMT, Jonathan Joo <j...@openjdk.org> wrote:
>> 8315149: Add hsperf counters for CPU time of internal GC threads > > Jonathan Joo has updated the pull request incrementally with one additional > commit since the last revision: > > Fix build issues Thanks for your continued work on this PR. I think it looks pretty good now once you fix the `long`/`jlong` issue mentioned by @dholmes-ora. And the solution proposed by @shipilev should work equally well on arm32. src/hotspot/share/gc/shared/stringdedup/stringDedupThread.cpp line 31: > 29: #include "runtime/handles.hpp" > 30: #include "runtime/os.hpp" > 31: #include "runtime/perfData.hpp" This include doesn't seem to be needed? ------------- PR Review: https://git.openjdk.org/jdk/pull/15082#pullrequestreview-1644233704 PR Review Comment: https://git.openjdk.org/jdk/pull/15082#discussion_r1337186614