RFR: 8320826: call allocate_shared_strings_array after all strings are interned

[Ioi Lam]

On Tue, 28 Nov 2023 21:21:43 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> `StringTable::allocate_shared_strings_array()` creates a Java object array that has the exact same size as the number of currently interned strings. Therefore, we need to move it to just before we enter the safepoint, where we know no more Java code would be execute.
>> 
>> I also had to change the assert in `StringTable::allocate_shared_strings_array()` to a condition check:  it's possible for `HeapShared::init_for_dumping()` to disable heap dumping when a Java agent transforms classes like jdk.internal.math.FDBigInteger.
>
> LGTM

Thanks @calvinccheung and @matias9927 for the review.

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

PR Comment: https://git.openjdk.org/jdk/pull/16839#issuecomment-1834407693