RFR: 8351194: Clean up Hotspot SA after 32-bit x86 removal [v6]

Chris Plummer cjplummer at openjdk.org
Tue Oct 28 19:40:49 UTC 2025


On Thu, 23 Oct 2025 13:05:49 GMT, Kerem Kat <krk at openjdk.org> wrote:

>> Remove 32-bit x86 specific code from the HotSpot Serviceability Agent following the removal of 32-bit x86 support.
>> 
>> - Removed x86-specific implementations and ifdef blocks.
>> - Renamed files with X86 in the name when they are also used from AMD64, e.g. `X86Frame` → `AMD64Frame`.
>> - Cleaned up platform detection logic in `PlatformInfo`.
>> - Updated documentation references.
>
> Kerem Kat has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains eight commits:
> 
>  - Merge branch 'master' into clean-x86-sa-JDK-8351194
>  - fix comments
>  - Revert "run RotateLeftNode*IdealizationTests on amd64 too"
>    
>    This reverts commit 1011b304f7cb4d195efc9239acd7784053c67cc1.
>  - requires hsdis
>  - run RotateLeftNode*IdealizationTests on amd64 too
>  - Merge branch 'master' into clean-x86-sa-JDK-8351194
>  - Merge branch 'master' into clean-x86-sa-JDK-8351194
>  - Clean up Hotspot SA after 32-bit x86 removal

Changes requested by cjplummer (Reviewer).

src/jdk.hotspot.agent/doc/hsdb.html line 34:

> 32:  <li> Class Browser - view Java classes, bytecode disassembly,
> 33:    or create .class files for selected classes
> 34:  <li> native disassembly (required hsdis) and nmethod disassembly with annotations for safepoint details.

Suggestion:

 <li> native disassembly (requires hsdis) and nmethod disassembly with annotations for safepoint details.

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

PR Review: https://git.openjdk.org/jdk/pull/27844#pullrequestreview-3390503752
PR Review Comment: https://git.openjdk.org/jdk/pull/27844#discussion_r2470817257


More information about the serviceability-dev mailing list