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