RFR: 8272107: Removal of Unsafe::defineAnonymousClass left a dangling C++ class [v2]

Harold Seigel hseigel at openjdk.java.net
Wed Aug 11 16:47:24 UTC 2021


On Wed, 11 Aug 2021 16:44:48 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

>> Please review this change to remove the now unused CPSlot class.  Uses of CPSlot have been replaced with Symbol*.  The change was tested by running JCK lang and VM tests, Mach5 tiers 1-2 on Linux, Mac OS, and WIndows and Macn5 tiers 3-5 on Linux x64.
>> 
>> Thanks, Harold
>
> Harold Seigel has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove slot_at()

Thanks Coleen, Ioi, and David for looking at this.
Please review this new commit that removes function slot_at() and, in most cases, replaces calls to slot_at() with calls to symbol_at().  Note that unresolved_strings functions need to call symbol_at_addr(), instead of symbol_at() because symbol_at() asserts if the cp tag is not utf8.

Thanks, Harold

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

PR: https://git.openjdk.java.net/jdk/pull/5070


More information about the hotspot-dev mailing list