# HG changeset patch
# User fyang
# Date 1497949207 -28800
# Node ID 5cebeef576b27ae0e3b8aec0dcc1f23fcde8b59c
# Parent  d01034a83ab2e76a95fe250ef9b74af332e1ebcd
8182581: aarch64: fix for crash caused by earlyret of compiled method
Summary: fix jvm crash caused by earlyret of compiled method for aarch64 port
Reviewed-by: aph
Contributed-by: snazarkin@azul.com

diff -r d01034a83ab2 -r 5cebeef576b2 hotspot/src/cpu/aarch64/vm/abstractInterpreter_aarch64.cpp
--- a/hotspot/src/cpu/aarch64/vm/abstractInterpreter_aarch64.cpp	Mon Jun 19 16:31:43 2017 -0700
+++ b/hotspot/src/cpu/aarch64/vm/abstractInterpreter_aarch64.cpp	Tue Jun 20 17:00:07 2017 +0800
@@ -109,9 +109,15 @@
   // for the callee's params we only need to account for the extra
   // locals.
   int size = overhead +
-         (callee_locals - callee_params)*Interpreter::stackElementWords +
+         (callee_locals - callee_params) +
          monitors * frame::interpreter_frame_monitor_size() +
-         temps* Interpreter::stackElementWords + extra_args;
+         // On the top frame, at all times SP <= ESP, and SP is
+         // 16-aligned.  We ensure this by adjusting SP on method
+         // entry and re-entry to allow room for the maximum size of
+         // the expression stack.  When we call another method we bump
+         // SP so that no stack space is wasted.  So, only on the top
+         // frame do we need to allow max_stack words.
+         (is_top_frame ? max_stack : temps + extra_args);
 
   // On AArch64 we always keep the stack pointer 16-aligned, so we
   // must round up here.