On Wed, 11 Mar 2026 11:52:30 GMT, Coleen Phillimore <[email protected]> wrote:
>> src/hotspot/share/runtime/interfaceSupport.cpp line 70:
>>
>>> 68: }
>>> 69: // Verify interpreter oopmap generation
>>> 70: if (!_no_async && GenerateOopMapALot) {
>>
>> Based on the description of GenerateOopMapALot, I would expect this to
>> execute even for the no-async case.
>
> But we don't - should I call it something different?
> I don't really like the double negative tbh.
Why not generate the oopmap based on GenerateOopMapALot only? I don't see the
reason for checking _no_async here.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30171#discussion_r2923353447
