RFR: 8355488: Add stress mode for C2 loop peeling [v3]

Marc Chevalier mchevalier at openjdk.org
Thu May 15 12:16:56 UTC 2025


On Tue, 13 May 2025 10:33:33 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Marc Chevalier has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Back to PRODUCT for consistency
>
> src/hotspot/share/opto/compile.cpp line 5295:
> 
>> 5293: 
>> 5294: uint& Compile::peeling_rounds_at_node(const Node* const head) {
>> 5295:   for(int i = 0; i < _peeling_rounds_of_node.length(); ++i) {
> 
> Suggestion:
> 
>   for (int i = 0; i < _peeling_rounds_of_node.length(); ++i) {

This code is gone.

> src/hotspot/share/opto/compile.cpp line 5297:
> 
>> 5295:   for(int i = 0; i < _peeling_rounds_of_node.length(); ++i) {
>> 5296:     auto& head_and_round_count = _peeling_rounds_of_node.at(i);
>> 5297:     if(head_and_round_count.first == head->_idx) {
> 
> Suggestion:
> 
>     if (head_and_round_count.first == head->_idx) {

This code is gone.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25140#discussion_r2091035138
PR Review Comment: https://git.openjdk.org/jdk/pull/25140#discussion_r2091035362


More information about the hotspot-compiler-dev mailing list