RFR: 8350616: Skip ValidateHazardPtrsClosure in non-debug builds [v2]

Aleksey Shipilev shade at openjdk.org
Wed Feb 26 09:21:01 UTC 2025


On Wed, 26 Feb 2025 07:12:17 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Simple cleanup to skip debug code in a non-debug build. This removes some overhead when working with very large numbers of threads. Basic perf testing starting a new thread when 20000 already exist:
>> 
>> Before: 532377 ns
>> After:  361178 ns
>> 
>> which is 32% improvement.
>> 
>> Testing:
>>   - tiers 1-3 sanity
>> 
>> Thanks
>
> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Change DEBUG to ASSERT

Wow, this is great! Change looks fine.

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

Marked as reviewed by shade (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23762#pullrequestreview-2643724883


More information about the hotspot-runtime-dev mailing list