RFR: 8359801: RISC-V: Simplify Interpreter::profile_taken_branch [v2]

Anjian Wen wenanjian at openjdk.org
Fri Jun 20 02:41:14 UTC 2025


On Fri, 20 Jun 2025 02:09:10 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Anjian Wen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   remove the preceding code comment
>
> src/hotspot/cpu/riscv/interp_masm_riscv.cpp line 1059:
> 
>> 1057:     // If no method data exists, go to profile_continue.
>> 1058:     // Otherwise, assign to mdp
>> 1059:     test_method_data_pointer(mdp, profile_continue);
> 
> Nit: Can you remove the preceding code comment like the other platforms?
> `// Otherwise, assign to mdp`

Thanks for the review, I have removed it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25848#discussion_r2157930633


More information about the hotspot-compiler-dev mailing list