RFR: 8315559: Delay TempSymbol cleanup to avoid symbol table churn [v4]

Coleen Phillimore coleenp at openjdk.org
Tue Nov 28 13:00:20 UTC 2023


On Tue, 28 Nov 2023 00:07:06 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> I don't remember why they were TempNewSymbol either but I don't think it matters for this test, they can be Symbol*.
>
> I still don't think it matters for the test, but this seems to be an effective workaround.

Rather than having new_stable_temp_symbol here, you could make them Symbol* like your previous change and then explicitly decrement the refcount for A, D and interf at the end of the test.  This seems like it would be cleaner and not have to interact with the SymbolTableCleaner code.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16398#discussion_r1407719187


More information about the hotspot-dev mailing list