RFR: 8301992: Embed SymbolTable CHT node

[Calvin Cheung]

On Wed, 15 Feb 2023 16:07:10 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> If the symbol is permanent, it's allocated to an arena not C heap but I agree that the variable name should be changed.
>
>> There's a version of NEW_RESOURCE_ARRAY that takes the Thread argument, so it doesn't have to call Thread::current()
> 
> I've changed it to:
> `u1* u1_buf = NEW_RESOURCE_ARRAY_IN_THREAD(current, u1, alloc_size);`

> The name of the `heap` parameter to this function should be changed. The node, which contains the Symbol, is always C_HEAP allocated. I would suggest renaming it to `is_permanent`. Same for the other functions that have a `heap` parameter.

Which other functions did you notice?

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

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