RFR 8173715 (rev2): Remove Flat Profiler

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Mon Aug 28 13:49:08 UTC 2017


> On compiler-dev Gerard specifically asks:
> 
> Can someone please review the 
> "hotspot/src/cpu/sparc/vm/macroAssembler_sparc.cpp” file to determine 
> whether we still need the three "save_frame_and_mov(0, Lmethod, 
> Lmethod);” calls where the comment said it was needed by -Xprof?
> 
> Do we leave those save_frame_and_mov() calls as is (and add comment as 
> to why they are still needed), or can we change them into save_frame() 
> calls?

I don't have a strong opinion, but I'd prefer to keep them as is.

Though all 3 usages are in debug code (either guarded by #ifdef ASSERT 
or VerifyThread), the code is transitively used in template interpreter.

So, I suspect dropping Lmethod will break interpreter stack frame 
attribution (frame::is_interpreted_frame_valid) in those narrow periods 
when temporary frames are alive and will affect JFR & AsyncGetCallTrace().

Best regards,
Vladimir Ivanov


More information about the hotspot-dev mailing list