RFR: 8317007: Add bulk removal of dead nmethods during class unloading [v7]

Fri Dec 15 14:05:59 UTC 2023

On Fri, 15 Dec 2023 13:57:03 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> src/hotspot/share/code/nmethod.cpp line 1476:
>> 
>>> 1474: 
>>> 1475:   CodeBlob::purge(free_code_cache_data, unregister_nmethod);
>>> 1476:   if (free_code_cache_data) {
>> 
>> This is effectively `true`.
>
> Please be more specific adding an actual suggestion. I have no idea what this statement means or what you expect here.

Oh, I got it. I meant to not have the assert at the start of the method. I'll remove it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16767#discussion_r1428007364