8189244: x86: eliminate frame::adjust_unextended_sp() overhead
authordlong
Fri, 13 Oct 2017 12:29:45 -0700
changeset 47625 60dae13727b8
parent 47624 b055cb5170f5
child 47626 52cf482c4d28
8189244: x86: eliminate frame::adjust_unextended_sp() overhead Reviewed-by: kvn
src/hotspot/cpu/x86/frame_x86.cpp
src/hotspot/cpu/x86/frame_x86.hpp
--- a/src/hotspot/cpu/x86/frame_x86.cpp	Fri Oct 13 15:08:56 2017 +0200
+++ b/src/hotspot/cpu/x86/frame_x86.cpp	Fri Oct 13 12:29:45 2017 -0700
@@ -383,6 +383,7 @@
 
 //------------------------------------------------------------------------------
 // frame::adjust_unextended_sp
+#ifdef ASSERT
 void frame::adjust_unextended_sp() {
   // On x86, sites calling method handle intrinsics and lambda forms are treated
   // as any other call site. Therefore, no special action is needed when we are
@@ -394,11 +395,12 @@
       // If the sender PC is a deoptimization point, get the original PC.
       if (sender_cm->is_deopt_entry(_pc) ||
           sender_cm->is_deopt_mh_entry(_pc)) {
-        DEBUG_ONLY(verify_deopt_original_pc(sender_cm, _unextended_sp));
+        verify_deopt_original_pc(sender_cm, _unextended_sp);
       }
     }
   }
 }
+#endif
 
 //------------------------------------------------------------------------------
 // frame::update_map_with_saved_link
--- a/src/hotspot/cpu/x86/frame_x86.hpp	Fri Oct 13 15:08:56 2017 +0200
+++ b/src/hotspot/cpu/x86/frame_x86.hpp	Fri Oct 13 12:29:45 2017 -0700
@@ -117,7 +117,7 @@
   // original sp we use that convention.
 
   intptr_t*     _unextended_sp;
-  void adjust_unextended_sp();
+  void adjust_unextended_sp() NOT_DEBUG_RETURN;
 
   intptr_t* ptr_at_addr(int offset) const {
     return (intptr_t*) addr_at(offset);