Integrated: 8322747: StringTable should be AllStatic

Kim Barrett kbarrett at openjdk.org
Sun Dec 31 17:29:45 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

This pull request has now been integrated.

Changeset: 518ec971
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/518ec9711411e6825668f72503a2e96824cd37ba
Stats:     6 lines in 1 file changed: 0 ins; 4 del; 2 mod

8322747: StringTable should be AllStatic

Reviewed-by: eosterlund

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

PR: https://git.openjdk.org/jdk/pull/17193


More information about the hotspot-runtime-dev mailing list