RFR: 8348117: The two-argument overload of SignatureHandlerLibrary::add is not used

Coleen Phillimore coleenp at openjdk.org
Tue Jan 21 12:24:35 UTC 2025


On Tue, 21 Jan 2025 08:44:56 GMT, David Holmes <dholmes at openjdk.org> wrote:

> The  `SignatureHandlerLibrary::add` two-arg API was added to support closed code under [JDK-8087333](https://bugs.openjdk.org/browse/JDK-8087333) in JDK 9. That closed code was removed again, still in JDK 9, as part of JEP 297 ([JDK-8168503](https://bugs.openjdk.org/browse/JDK-8168503)) which unified the closed and open ARM ports - leaving the API unused.
> 
> There were other leftovers from the same set of changes so they are cleaned up as well - nothing significant, just unnecessary friend declarations, comments, and one other method.
> 
> Testing:
>  - all builds in tiers 1-5
> 
> Thanks.

Looks good!

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

Marked as reviewed by coleenp (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23207#pullrequestreview-2564461281


More information about the hotspot-dev mailing list