On Mon, 28 Oct 2024 21:01:47 GMT, Patricio Chilano Mateo 
<pchilanom...@openjdk.org> wrote:

>> I tried to track down how interpreter_frame_num_oops() is used, and as far 
>> as I can tell, it is only used to compare against the bitmap in debug/verify 
>> code.  So if this slot was added here, shouldn't there be a corresponding 
>> change for the bitmap?
>
> When creating the bitmap, processing oops in an interpreter frame is done 
> with `frame::oops_interpreted_do()` which already counts this extra oop for 
> native methods.

What are we counting now with MaskFillerForNativeFrame that we weren't counting 
before this change?  in MaskFillerForNative::set_one.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1819869538

Reply via email to