Request for reviews (M): 6833129 : specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot

[John Rose]

On Jul 6, 2009, at 5:21 PM, changpeng fang - Sun Microsystems - Santa  
Clara United States wrote:

> http://cr.openjdk.java.net/~cfang/6833129/webrev.01/

Here's a nit:  is_restart() in scopeDesc.hpp should be a const function.

I'd like to see comment in each case demonstrating why that the values  
captured in dummy_restart, and the throwaway restart in  
javaClasses.cpp (which should be called dummy_restart also) don't  
matter.

I'd still like to see the restart decision made by continuation_for  
turned into an assert.  At least compare it with is_restart():

>    pc = Interpreter::continuation_for(method(), bcp,  
> callee_parameters, is_top_frame, use_next_mdp);
> + assert(is_restart() == !use_next_mdp, "");

If the assert fails, there may be a bug.

-- John