RFR(S): 8136461: PhaseIdealLoop::try_move_store_before_loop() may bypass early loop exit

[Vladimir Kozlov]

Good. You should have done it in original 8080289 changes.

Thanks,
Vladimir

On 9/14/15 6:04 AM, Roland Westrelin wrote:
> http://cr.openjdk.java.net/~roland/8136461/webrev.00/
>
> In some cases, a Store that is moved out of a loop by a try_move_store_before_loop(), is input to a MergeMem that itself is input to a bottom memory Phi for the loop. The loop has an exit point before the Store. That exit point uses the bottom memory Phi as memory state. Moving the Store out of the loop shouldn't be attempted in that case. I turned the verification code in that method (that found that problem) into an extra validity check.
>
> Roland.
>