RFR: 8348117: The two-argument overload of SignatureHandlerLibrary::add is not used
David Holmes
dholmes at openjdk.org
Tue Jan 21 21:49:38 UTC 2025
On Tue, 21 Jan 2025 12:22:26 GMT, Coleen Phillimore <coleenp 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!
Thanks for the reviews @coleenp and @shipilev !
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23207#issuecomment-2605805194
More information about the hotspot-dev
mailing list