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

Yangfei (Felix) felix.yang at huawei.com
Sat Oct 12 01:58:55 UTC 2019


>> 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


More information about the hotspot-compiler-dev mailing list