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

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Mar 24 16:20:11 UTC 2017

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.


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

More information about the hotspot-compiler-dev mailing list