RFR; 8240223: Use consistent predicate order in and with PhaseIdealLoop::find_predicate

Stefan Karlsson stefan.karlsson at oracle.com
Mon Mar 2 09:24:24 UTC 2020


Thanks, Nils.

StefanK

On 2020-02-28 14:21, Nils Eliasson wrote:
> +1
> 
> // Nils
> 
> On 2020-02-28 14:03, Tobias Hartmann wrote:
>> Hi Stefan,
>>
>> looks good to me. Since you are touching that code anyway, could you 
>> please remove the whitespace
>> after NULL in loopnode.cpp lines 2661, 2668, 2674? (no new webrev 
>> required)?
>>
>> Thanks,
>> Tobias
>>
>> On 28.02.20 12:12, Stefan Karlsson wrote:
>>> Hi all,
>>>
>>> Please review this patch to make the predicate search order 
>>> consistent in
>>> PhaseIdealLoop::find_predicate and some of its usages.
>>>
>>> https://cr.openjdk.java.net/~stefank/8240223/webrev.01/
>>> https://bugs.openjdk.java.net/browse/JDK-8240223
>>>
>>> The code wasn't broken before, but making the search order 
>>> consistent, make it easier for the casual
>>> reader to more easily understand the code related to the loop 
>>> predicates.
>>>
>>> Minimal testing done, will run through tier1-3
>>>
>>> Thanks,
>>> StefanK
> 


More information about the hotspot-compiler-dev mailing list