This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch,avr] Fix PR60300: Minor prologue improvement.
- From: Denis Chertykov <chertykov at gmail dot com>
- To: Georg-Johann Lay <avr at gjlay dot de>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 22 Nov 2016 21:23:04 +0300
- Subject: Re: [patch,avr] Fix PR60300: Minor prologue improvement.
- Authentication-results: sourceware.org; auth=none
- References: <e112ba96-8ef6-f3d8-2b9a-f1f93fdfca27@gjlay.de>
2016-11-22 15:41 GMT+03:00 Georg-Johann Lay <avr@gjlay.de>:
> This patch is a minor improvement of prologue length. It now allows frame
> sizes of up to 11 to be allocated by RCALL + PUSH 0 sequences but limits the
> number of RCALLs to 3.
>
> The PR has some discussion on size vs. speed consideration w.r. to using
> RCALL in prologues, and following that I picked the rather arbitrary upper
> bound of 3 RCALLs. The prior maximal frame size opt to such sequences was 6
> which also never produced more than 3 RCALLs.
>
> Ok for trunk?
>
>
> Johann
>
> gcc/
> PR target/60300
> * config/avr/constraints.md (Csp): Widen range to [-11..6].
> * config/avr/avr.c (avr_prologue_setup_frame): Limit number
> of RCALLs in prologue to 3.
Approved.