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

Aleksey Shipilev shade at openjdk.org
Thu Nov 9 18:14:08 UTC 2023


On Thu, 9 Nov 2023 16:52:26 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Good idea, changed to 128. I'm no expert but the code looks better for it here: https://godbolt.org/z/7cYv9Pzz7.
>
> I like it!

A common thing is to do the mod conversion right here, without relying on optimizing compilers:
`% CLEANUP_DELAY_MAX_ENTRIES` -> `& (CLEANUP_DELAY_MAX_ENTRIES - 1)`.

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

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


More information about the hotspot-dev mailing list