review for 6990212: JSR 292 JVMTI MethodEnter hook is not called for JSR 292 bootstrap and target methods
tom.rodriguez at oracle.com
Mon Jul 11 11:01:13 PDT 2011
On Jul 11, 2011, at 9:18 AM, Christian Thalinger wrote:
> 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:
You're right. The original code was more clever and in simplifying it I reversed it. I realized after Vladimir's comment that I hadn't run my tests on sparc, though I would have sworn I had. Anyway, I'm going to make it look more like the original code and run my tests to make sure it's all working properly on sparc. Thanks!
> + 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
More information about the hotspot-compiler-dev