[9] RFR(XS): 8154763: Crash with "assert(RangeCheckElimination)" if RangeCheckElimination is disabled

[Vladimir Kozlov]

Looks good.

Thanks,
Vladimir

On 4/21/16 6:56 AM, Tobias Hartmann wrote:
> Hi,
>
> please review the following patch:
>
> https://bugs.openjdk.java.net/browse/JDK-8154763
> http://cr.openjdk.java.net/~thartmann/8154763/webrev.00/
>
> JDK-8151573 introduced multiversioning for range check elimination. Explicitly turning of range check elimination now crashes the VM with an assert in PhaseIdealLoop::has_range_checks() because we assume that this is only called if range check elimination is enabled.
>
> I think we should disable multiversioning if range check elimination is turned off. I added the corresponding check.
>
> Tested with regression test and RBT (running).
>
> Thanks,
> Tobias
>