Request for reviews (XS): 6990933: assert(sender_cb) failed: sanity in frame::sender_for_interpreter_frame

[Tom Rodriguez]

Looks good.

tom

On Dec 16, 2010, at 6:57 AM, Christian Thalinger wrote:

> http://cr.openjdk.java.net/~twisti/6990933/webrev.01/
> 
> 6990933: assert(sender_cb) failed: sanity in frame::sender_for_interpreter_frame
> Reviewed-by:
> 
> The bug is that the nmethod constructor uses
> nmethod::has_method_handle_invokes but the _has_method_handle_invokes
> flag is not initialized until nmethod::copy_scopes_pcs is called and
> thus always returns false.  This results in _deoptimize_mh_offset to
> be -1 and so deopt_mh_handler_begin points 1 byte before the nmethod
> pointer resulting in the assert.