On Mon, 25 May 2026 06:02:36 GMT, Yasumasa Suenaga <[email protected]> wrote:

>> Benjamin Peterson has updated the pull request with a new target base due to 
>> a merge or a rebase. The incremental webrev excludes the unrelated changes 
>> brought in by the merge/rebase. The pull request contains three additional 
>> commits since the last revision:
>> 
>>  - use space instead of ::
>>  - Merge branch 'master' into perfmap-stub-entries
>>  - 8383709: Linux perf map should record individual stubs
>>    
>>    StubRoutines blobs bundle many individually-named stubs into a single 
>> BufferBlob, so perf profiles previously attributed all samples inside those 
>> blobs to the enclosing blob name (e.g. "StubRoutines (compiler stubs)"). 
>> Walk the StubCodeDesc list and emit a line per descriptor, so perf can name 
>> the actual stub.
>
> src/hotspot/share/code/codeCache.cpp line 1980:
> 
>> 1978:     CodeBlob *cb = iter.method();
>> 1979:     if (is_stub_code_blob(cb)) {
>> 1980:       continue;
> 
> Please add some comments that stub code would be handled latter.

added

      // Individual stub routines are dumped after the main loop.

here

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31007#discussion_r3298931262

Reply via email to