This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Remove support for INITIAL_FRAME_POINTER_OFFSET

From: Jeff Law <law at redhat dot com>
To: Bernd Edlinger <bernd dot edlinger at hotmail dot de>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 9 Sep 2016 16:12:20 -0600
Subject: Re: [PATCH] Remove support for INITIAL_FRAME_POINTER_OFFSET
Authentication-results: sourceware.org; auth=none
References: <AM4PR0701MB21621AFDFE6AA2D9F4D987D7E4FA0@AM4PR0701MB2162.eurprd07.prod.outlook.com>

On 09/09/2016 03:40 PM, Bernd Edlinger wrote:

Hi,

I think it is time to remove support for INITIAL_FRAME_POINTER_OFFSET, which is no longer
used by any target today.  This removes a bunch of conditional code, and fixes a few bits
in the documentation.  I'd say that part of the documentation is quite out of sync, but I just
have to stop somewhere.


Bootstrapped and reg-tested on x86_64-pc-linux.gnu
Is it OK for trunk?


Thanks
Bernd.


changelog-initial-frame-pointer-offset.txt


2016-09-09  Bernd Edlinger  <bernd.edlinger@hotmail.de>

	* doc/tm.texi.in (INITIAL_FRAME_POINTER_OFFSET): Remove.
	(ELIMINABLE_REGS, TARGET_CAN_ELIMINATE,
	INITIAL_ELIMINATION_OFFSET) : Update documentation.
	* target.def (frame_pointer_required, can_eliminate): Likewise.
	* doc/tm.texi: Regenerated.
	* builtins.c (expand_builtin_setjmp_receiver): Remove #ifdef
	ELIMINABLE_REGS.
	* df-scan.c (df_hard_reg_init): Likewise.
	* ira.c (ira_setup_eliminable_regset): Likewise.
	* lra-eliminations.c (reg_eliminate_1, (update_reg_eliminate,
	init_elim_table): Likewise.
	* reload1.c (reg_eliminate_1, verify_initial_elim_offsets,
	set_initial_elim_offsets, update_eliminables,
	init_elim_table): Likewise.
	* rtlanal.c (get_initial_register_offset): Likewise.
	* config/ft32/ft32.h (INITIAL_FRAME_POINTER_OFFSET): Remove.
	* config/m32r/m32r.h (INITIAL_FRAME_POINTER_OFFSET): Remove.
	* config/moxie/moxie.h (INITIAL_FRAME_POINTER_OFFSET): Remove.
	* config/vax/vax.h (INITIAL_FRAME_POINTER_OFFSET): Remove.

Note that INITIAL_FRAME_POINTER_OFFSET shows up in comments in variousports. Can you fix those as well.



OK with that change.

Jeff

References:
- [PATCH] Remove support for INITIAL_FRAME_POINTER_OFFSET
  - From: Bernd Edlinger

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]