|
 |
ee890f |
2014-11-04 Jiong Wang <jiong.wang@arm.com>
|
|
|
ee890f |
Wilco Dijkstra <wilco.dijkstra@arm.com>
|
|
|
ee890f |
|
|
|
ee890f |
PR target/63293
|
|
|
ee890f |
* config/aarch64/aarch64.c (aarch64_expand_epiloue): Add barriers before
|
|
|
ee890f |
stack adjustment.
|
|
|
ee890f |
|
|
|
ee890f |
--- gcc/config/aarch64/aarch64.c (revision 217090)
|
|
|
ee890f |
+++ gcc/config/aarch64/aarch64.c (revision 217091)
|
|
|
ee890f |
@@ -1989,6 +1989,9 @@ aarch64_expand_epilogue (bool for_sibcal
|
|
|
ee890f |
rtx insn;
|
|
|
ee890f |
rtx cfa_reg;
|
|
|
ee890f |
rtx cfi_ops = NULL;
|
|
|
ee890f |
+ /* We need to add memory barrier to prevent read from deallocated stack. */
|
|
|
ee890f |
+ bool need_barrier_p = (get_frame_size () != 0
|
|
|
ee890f |
+ || cfun->machine->saved_varargs_size);
|
|
|
ee890f |
|
|
|
ee890f |
aarch64_layout_frame ();
|
|
|
ee890f |
original_frame_size = get_frame_size () + cfun->machine->saved_varargs_size;
|
|
|
ee890f |
@@ -2030,6 +2033,9 @@ aarch64_expand_epilogue (bool for_sibcal
|
|
|
ee890f |
if (frame_pointer_needed
|
|
|
ee890f |
&& (crtl->outgoing_args_size || cfun->calls_alloca))
|
|
|
ee890f |
{
|
|
|
ee890f |
+ if (cfun->calls_alloca)
|
|
|
ee890f |
+ emit_insn (gen_stack_tie (stack_pointer_rtx, stack_pointer_rtx));
|
|
|
ee890f |
+
|
|
|
ee890f |
insn = emit_insn (gen_add3_insn (stack_pointer_rtx,
|
|
|
ee890f |
hard_frame_pointer_rtx,
|
|
|
ee890f |
GEN_INT (- fp_offset)));
|
|
|
ee890f |
@@ -2048,6 +2054,9 @@ aarch64_expand_epilogue (bool for_sibcal
|
|
|
ee890f |
/* Restore the frame pointer and lr if the frame pointer is needed. */
|
|
|
ee890f |
if (offset > 0)
|
|
|
ee890f |
{
|
|
|
ee890f |
+ if (need_barrier_p && (!frame_pointer_needed || !fp_offset))
|
|
|
ee890f |
+ emit_insn (gen_stack_tie (stack_pointer_rtx, stack_pointer_rtx));
|
|
|
ee890f |
+
|
|
|
ee890f |
if (frame_pointer_needed)
|
|
|
ee890f |
{
|
|
|
ee890f |
rtx mem_fp, mem_lr;
|
|
|
ee890f |
@@ -2067,6 +2076,10 @@ aarch64_expand_epilogue (bool for_sibcal
|
|
|
ee890f |
+ UNITS_PER_WORD));
|
|
|
ee890f |
emit_insn (gen_load_pairdi (reg_fp, mem_fp, reg_lr, mem_lr));
|
|
|
ee890f |
|
|
|
ee890f |
+ if (need_barrier_p)
|
|
|
ee890f |
+ emit_insn (gen_stack_tie (stack_pointer_rtx,
|
|
|
ee890f |
+ stack_pointer_rtx));
|
|
|
ee890f |
+
|
|
|
ee890f |
insn = emit_insn (gen_add2_insn (stack_pointer_rtx,
|
|
|
ee890f |
GEN_INT (offset)));
|
|
|
ee890f |
}
|
|
|
ee890f |
@@ -2128,6 +2141,9 @@ aarch64_expand_epilogue (bool for_sibcal
|
|
|
ee890f |
|
|
|
ee890f |
if (frame_size > -1)
|
|
|
ee890f |
{
|
|
|
ee890f |
+ if (need_barrier_p)
|
|
|
ee890f |
+ emit_insn (gen_stack_tie (stack_pointer_rtx, stack_pointer_rtx));
|
|
|
ee890f |
+
|
|
|
ee890f |
if (frame_size >= 0x1000000)
|
|
|
ee890f |
{
|
|
|
ee890f |
rtx op0 = gen_rtx_REG (Pmode, IP0_REGNUM);
|