On Wed, 6 May 2026 08:24:33 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/cpu/arm/c1_MacroAssembler_arm.cpp line 263:
> 
>> 261: 
>> 262: void C1_MacroAssembler::save_profile_rng() {
>> 263:   if (ProfileCaptureRatio != 1) {
> 
> Here and later, looks like a common way to test if `PCR` is enabled is `PCR > 
> 0`, use it everywhere?

Done.

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

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

Reply via email to