RFR: 8311087: PhiNode::wait_for_region_igvn should break early [v2]

Johan Sjölen jsjolen at openjdk.org
Tue Jul 4 10:35:21 UTC 2023


On Fri, 30 Jun 2023 06:29:02 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Wrap continue in braces
>
> src/hotspot/share/opto/cfgnode.cpp line 1944:
> 
>> 1942:     Node* n = in(j);
>> 1943: 
>> 1944:     if (rc == nullptr || !rc->is_Proj()) continue;
> 
> Maybe you could put braces around the `continue` statements.

Sounds good, I didn't change indentation though.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14707#discussion_r1251838476


More information about the hotspot-compiler-dev mailing list