RFR(M): 8229499: Node budget assert in fuzzed test

Nils Eliasson nils.eliasson at oracle.com
Mon Sep 23 19:30:44 UTC 2019


+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


More information about the hotspot-compiler-dev mailing list