review for 6990212: JSR 292 JVMTI MethodEnter hook is not called for JSR 292 bootstrap and target methods

[Christian Thalinger]

On Jul 11, 2011, at 5:43 PM, Christian Thalinger wrote:
> On Jul 9, 2011, at 12:21 AM, Tom Rodriguez wrote:
>> Coleen point out that it's confusing to reuse the name jump_from_interpreted since we're not really in the interpreter.  I've changed it to jump_from_method_handle and left that note that it parallels jump_from_interpreted.
> 
> This looks good.

Looking at it again I think the SPARC code is wrong:

+     const Address interp_only(G2_thread, JavaThread::interp_only_mode_offset());
+     __ ld(interp_only, temp);
+     __ tst(temp);
+     __ br(Assembler::notZero, true, Assembler::pn, skip_compiled_code);
+     __ delayed()->nop();
+     __ jump_indirect_to(Address(method, methodOopDesc::interpreter_entry_offset()), temp);
+     __ bind(skip_compiled_code);

It should be:

+     __ br(Assembler::zero, false, Assembler::pt, run_compiled_code);

-- Christian