On Thu, 11 Aug 2022 07:12:17 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
> Resize the hash table to minimize collisions. The correct size was already > calculated, but was not used. Looks good. For some historical perspective, it looks like this change came with the following commit: https://github.com/openjdk/jdk/commit/2841c5eb2b502f7b299347a203df52ab918cadce#diff-166f525c275cf2d9812ceb9f8c9c98352974e23a898df4d71762d9c7d0dc2481 Oddly that is for the initial push of AOT support. My first guess was that AOT greatly increased the number of symbols, and maybe some performance profiling picked up this hash table performance, but that seems unlikely since I see no reason why AOT would have resulted in profiling SA support. ------------- Marked as reviewed by cjplummer (Reviewer). PR: https://git.openjdk.org/jdk/pull/9834
