On Tue, 25 Nov 2025 20:33:48 GMT, Alex Menkov <[email protected]> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> review: fix typo in a EATests.java comment
>
> src/hotspot/share/prims/jvmtiThreadState.cpp line 707:
>
>> 705: for (int idx = 0; idx < deopts->length(); idx++) {
>> 706: int frame_number = deopts->at(idx);
>> 707: deopts->remove_at(idx);
>
> The code forward iterates the array removing the entries? it will skip every
> other element (indexes change after removal)
Nice catch, thanks! It is easy to forget about it. Fixed now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28407#discussion_r2561806700
