RFR(XS): 8134288 compiler/runtime/6859338/Test6859338.java crashes in PhaseIdealLoop::try_move_store_after_loop

Roland Westrelin roland.westrelin at oracle.com
Fri Aug 28 13:15:08 UTC 2015


Thanks for the review, Vladimir.

Roland.

> On Aug 26, 2015, at 6:33 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
> Looks fine.
> 
> Thanks,
> Vladimir
> 
> On 8/26/15 4:31 AM, Roland Westrelin wrote:
>> http://cr.openjdk.java.net/~roland/8134288/webrev.00/
>> 
>> Stores from code generated by c2 to update profiling (profile_taken_branch() called from Parse::do_if() if ProfileInterpreter is off)) doesn’t have a control. This looks like a corner case so I went for the simplest fix and excluded stores with no controls from the logic that tries to move stores out of loops.
>> 
>> Roland.
>> 



More information about the hotspot-compiler-dev mailing list