This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Provide default for FRAME_POINTER_REQUIRED
- From: Ian Lance Taylor <iant at google dot com>
- To: Anatoly Sokolov <aesok at post dot ru>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sat, 02 May 2009 16:52:06 -0700
- Subject: Re: [PATCH] Provide default for FRAME_POINTER_REQUIRED
- References: <127448375.20090502145630@post.ru>
Anatoly Sokolov <aesok@post.ru> writes:
> This patch deletes FRAME_POINTER_REQUIRED for ports that have it defined as 0.
> I propose this patch for simplify hookize the FRAME_POINTER_REQUIRED macro.
>
> The patch has been bootstrapped on i686-pc-linux-gnu.
>
> Ok?
>
> Anatoly.
>
>
> 2008-04-02 Anatoly Sokolov <aesok@post.ru>
>
> * defaults.h (FRAME_POINTER_REQUIRED): Provide default.
> * doc/tm.texi (FRAME_POINTER_REQUIRED): Revise documentation.
> * config/alpha/alpha.h (FRAME_POINTER_REQUIRED): Delete.
> * config/s390/s390.h (FRAME_POINTER_REQUIRED): Delete.
> * config/spu/spu.h (FRAME_POINTER_REQUIRED): Delete.
> * config/sh/sh.h (FRAME_POINTER_REQUIRED): Delete.
> * config/pdp11/pdp11.h (FRAME_POINTER_REQUIRED): Delete.
> * config/stormy16/stormy16.h (FRAME_POINTER_REQUIRED): Delete.
> * config/m68hc11/m68hc11.h (FRAME_POINTER_REQUIRED): Delete.
> * config/iq2000/iq2000.h (FRAME_POINTER_REQUIRED): Delete.
> * config/mn10300/mn10300.h (FRAME_POINTER_REQUIRED): Delete.
> * config/ia64/ia64.h (FRAME_POINTER_REQUIRED): Delete.
> * config/m68k/m68k.h (FRAME_POINTER_REQUIRED): Delete.
> * config/rs6000/rs6000.h (FRAME_POINTER_REQUIRED): Delete.
> * config/picochip/picochip.h (FRAME_POINTER_REQUIRED): Delete.
> * config/mcore/mcore.h (FRAME_POINTER_REQUIRED): Delete.
> * config/h8300/h8300.h (FRAME_POINTER_REQUIRED): Delete.
> * config/v850/v850.h (FRAME_POINTER_REQUIRED): Delete.
This patch is OK if you are going to continue by hookizing the macro as
you suggest. If you aren't going to continue, then some of the existing
definitions need to be changed from int to bool.
Thanks.
Ian