On Wed, 6 May 2026 08:21:59 GMT, Aleksey Shipilev <[email protected]> wrote:

>> Andrew Haley has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains 111 commits:
>> 
>>  - Merge branch 'master' into JDK-8134940
>>  - Review comments
>>  - Review comments
>>  - Add FrameMap::last_fpu_reg()
>>  - Review comments
>>  - Review comments
>>  - Review comments
>>  - Merge branch 'JDK-8134940' of https://github.com/theRealAph/jdk into 
>> JDK-8134940
>>  - More
>>  - Merge branch 'JDK-8134940' of https://github.com/theRealAph/jdk into 
>> JDK-8134940
>>  - ... and 101 more: https://git.openjdk.org/jdk/compare/3e1d0b8e...4610777b
>
> src/hotspot/share/c1/c1_LIR.cpp line 897:
> 
>> 895:     }
>> 896: 
>> 897:     case lir_increment_counter: {
> 
> Just to make sure I understand: `freq_op` operand is missing here 
> intentionally, or?

Yes. The only possible type for `freq_op` is a jint. I guess it could in theory 
become something else one day, but it very likely won't.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28541#discussion_r3335995208

Reply via email to