[8u, 9] RFR(S): 8177095: Range check dependent CastII/ConvI2L is prematurely eliminated

[Tobias Hartmann]

Hi Vladimir,

On 23.03.2017 17:10, Vladimir Kozlov wrote:
>> I'll do a performance run to check if there is a regression.
> 
> Okay.

The results are ready (see link in bug report). There are no regressions for those benchmarks,

> You need to look on code in PhaseIdealLoop::insert_pre_post_loops() for split iterations names.
> 
> We use next terminology.
> "old loop" is original loop before iterations split.
> After split we have:
> "pre-loop", "main-loop", "post-loop". And Michael B. added an other vector-post-loop for vectorization in jdk 9.

Thanks, Vladimir. I changed the terminology in the test comments from "old loop" to "main loop".

If there are no objections, I'll push this as soon as my 'jdk9-fix-request' is approved.

Best regards,
Tobias