RFR(S): 8231988: Unexpected test result caused by C2 IdealLoopTree::do_remove_empty_loop

[Tobias Hartmann]

Hi Felix,

looks good.

Thanks,
Tobias

On 11.10.19 18:58, Yangfei (Felix) wrote:
>>> Hi Felix,
>>
>> On 11.10.19 02:45, Yangfei (Felix) wrote:
>>> Updated webrev: http://cr.openjdk.java.net/~fyang/8231988/webrev.01/
>>
>> If possible, tests should have a name that describes their purpose and should
>> be put into one of the corresponding folders. In this case, I would suggest
>> TestRemoveEmptyLoop and put it into compiler/loopopts/.
>>
> 
> Hi Tobias,
> 
>     That sounds good.  Updated webrev: http://cr.openjdk.java.net/~fyang/8231988/webrev.02/
>     Newly added test pass on aarch64 & x64 platform.  Is it OK?
> 
> Thanks,
> Felix
>