RFR(XXS): 8230185 - assert(is_Loop()) failed: invalid node class

[Bhateja, Jatin]

Hi Tobias,

Please find updated patch at following link.

http://cr.openjdk.java.net/~jbhateja/8230185/webrev.03/

Thanks,
Jatin

> -----Original Message-----
> From: Tobias Hartmann <tobias.hartmann at oracle.com>
> Sent: Tuesday, December 17, 2019 1:10 PM
> To: Bhateja, Jatin <jatin.bhateja at intel.com>
> Cc: hotspot-compiler-dev at openjdk.java.net; Vladimir Kozlov
> <vladimir.kozlov at oracle.com>
> Subject: Re: RFR(XXS): 8230185 - assert(is_Loop()) failed: invalid node class
> 
> Hi Jatin,
> 
> On 16.12.19 16:42, Bhateja, Jatin wrote:
> > Please find below the updated patch with the test case.
> >
> > http://cr.openjdk.java.net/~jbhateja/8230185/webrev.02/
> 
> Thanks for adding the test. Some comments:
> - We try to avoid bug ids in test names. I would suggest a more descriptive
> name like "TestIrreducibleLoopWithVNNI".
> - Please also add the test to package compiler.loopopts
> - For Java code, we use 4 whitespace indentation
> - The variable 'c' is not used in 'mainTest'
> 
> Thanks,
> Tobias