[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-compiler-dev
Subject:    Re: RFR(M): 8229499: Node budget assert in fuzzed test
From:       Patric Hedlin <patric.hedlin () oracle ! com>
Date:       2019-09-26 7:15:19
Message-ID: 6ad6d653-eac8-29be-fe32-3a31fadd56a2 () oracle ! com
[Download RAW message or body]

Thanks for reviewing Tobias & Nils.

/Patric

On 2019-09-23 21:30, Nils Eliasson wrote:
> +1
>
> Reviewed.
>
> // Nils
>
> On 2019-09-20 10:43, Tobias Hartmann wrote:
>> Hi Patric,
>>
>> this looks reasonable to me. Nice cleanup of related code.
>>
>> Just noticed an excess whitespace in the comment in loopnode.cpp:2503.
>>
>> Best regards,
>> Tobias
>>
>> On 18.09.19 17:53, Patric Hedlin wrote:
>>> Dear all,
>>>
>>> I would like to ask for help to review the following change/update:
>>>
>>> Issue:   https://bugs.openjdk.java.net/browse/JDK-8229499
>>> Webrev: http://cr.openjdk.java.net/~phedlin/tr8229499/
>>>
>>> 8229499: Node budget assert in fuzzed test
>>>
>>>          Refactored (full) loop unroll estimate to approximate complex 
>>> loop cases
>>>          more accurately (similar to cloning). Same ad-hoc. (Minor 
>>> cleanup in the
>>>          related dump-support.)
>>>
>>>
>>> Testing: hs-tier1..3, hs-precheckin-comp
>>>          Including new TC
>>>
>>>
>>> Best regards,
>>> Patric
[prev in list] [next in list] [prev in thread] [next in thread]