RFR: 8322747: StringTable should be AllStatic

Erik Österlund eosterlund at openjdk.org
Wed Dec 27 19:38:40 UTC 2023


On Wed, 27 Dec 2023 03:52:49 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this trivial change to StringTable, making it derive from
> AllStatic rather than CHeapObj<mtSymbol>, to match the implementation.
> 
> While I was there, also did a little cleanup of related forward and friend
> declarations.  StringTableCreateEntry doesn't exist.  Don't need forward
> declaration for StringTable right before it's definition.  StringTable doesn't
> need to befriend Symbol.
> 
> Testing: mach5 tier1

Marked as reviewed by eosterlund (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/17193#pullrequestreview-1797465736


More information about the hotspot-runtime-dev mailing list