Request for reviews (XXS): 6868269 CompileTheWorld assertion failure introduced by the reexecute bit implementation

[Vladimir Kozlov]

Good.

Vladimir

changpeng fang - Sun Microsystems - Santa Clara United States wrote:
> http://cr.openjdk.java.net/~cfang/6868269/webrev.00/
> 
> Problem: For inline_native_clone, there are two exceptions with 
> different reexecute states.
> This caused the assertion on JVMState::same_calls_as
> 
> Solution: JVMState::same_calls_as was designed to verify the same 
> method/bci pairs. The current
> implementation of reexecute logic does not require the same method/bci 
> to have a single reexecute
> state, i.e. the implementer could change the reexecute state during the 
> parsing of a bci.
> 
> So the solution is simply remove the following line in 
> JVMState::same_calls_as:
> 
> -     if (p->_reexecute != q->_reexecute)  return false;
> 
> 
> 
> Tests: JPRT, CompileTheWorld!
> 
> 
> 
> Thanks,
> 
> 
> Changpeng
>