This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] xtensa: Fix PR target/78118
- From: "augustine dot sterling at gmail dot com" <augustine dot sterling at gmail dot com>
- To: Max Filippov <jcmvbkbc at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "linux-xtensa at linux-xtensa dot org" <linux-xtensa at linux-xtensa dot org>
- Date: Tue, 1 Nov 2016 09:28:43 -0700
- Subject: Re: [PATCH] xtensa: Fix PR target/78118
- Authentication-results: sourceware.org; auth=none
- References: <1477981952-8738-1-git-send-email-jcmvbkbc@gmail.com>
On Mon, Oct 31, 2016 at 11:32 PM, Max Filippov <jcmvbkbc@gmail.com> wrote:
> 2016-10-26 Max Filippov <jcmvbkbc@gmail.com>
> gcc/
> * config/xtensa/xtensa-protos.h
> (xtensa_use_return_instruction_p): New prototype.
> * config/xtensa/xtensa.c (xtensa_current_frame_size,
> xtensa_callee_save_size): Remove.
> (struct machine_function): Add new fields: current_frame_size,
> callee_save_size, frame_laid_out and epilogue_done.
> (compute_frame_size, xtensa_expand_prologue,
> xtensa_expand_epilogue): Replace xtensa_callee_save_size with
> cfun->machine->callee_save_size and xtensa_current_frame_size
> with cfun->machine->current_frame_size.
> (compute_frame_size): Update cfun->machine->frame_laid_out and
> don't update frame layout after reload completion.
> (xtensa_expand_epilogue): Set cfun->machine->epilogue_done
> instead of zeroing xtensa_current_frame_size.
> (xtensa_use_return_instruction_p): New function.
> * config/xtensa/xtensa.h (xtensa_current_frame_size): Remove
> declaration.
> (INITIAL_ELIMINATION_OFFSET): Use return value of
> compute_frame_size instead of xtensa_current_frame_size value.
> * config/xtensa/xtensa.md ("return" pattern): Use new predicate
> function xtensa_use_return_instruction_p instead of inline code.
Approved. Please apply.