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

[Vladimir Kozlov]

On 3/23/17 11:40 PM, Tobias Hartmann wrote:
> 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,

Good. I approved the fix.

Thanks,
Vladimir

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