This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH/RFA] ARM -- Implement ATPCS stack alignment rules
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Jason R Thorpe <thorpej at wasabisystems dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Richard dot Earnshaw at arm dot com
- Date: Thu, 07 Nov 2002 13:38:00 +0000
- Subject: Re: [PATCH/RFA] ARM -- Implement ATPCS stack alignment rules
- Organization: ARM Ltd.
- Reply-to: Richard dot Earnshaw at arm dot com
> On Tue, Nov 05, 2002 at 08:12:34AM -0800, Jason R Thorpe wrote:
>
> > OK, I've folded this in. Lemme try and track down the source of these
> > regressions, and I'll post another patch...
>
> Ok, the source of the Thumb regressions has been found (thanks a million,
> Richard :-), and an arm-elf sim testsuite run is in-progress.
>
> OK for mainline if it passes?
>
> 2002-11-06 Jason Thorpe <thorpej@wasabisystems.com>
>
> * config/arm/arm-protos.h (arm_get_frame_size)
> (thumb_get_frame_size): New prototypes.
> * config/arm/arm.c (arm_get_frame_size)
> (thumb_get_frame_size): New functions.
> (use_return_insn, arm_output_epilogue, arm_output_function_epilogue)
> (arm_compute_initial_elimination_offset, arm_expand_prologue): Use
> arm_get_frame_size.
> (thumb_expand_prologue, thumb_expand_epilogue): Use
> thumb_get_frame_size.
> * config/arm/arm.h (PREFERRED_STACK_BOUNDARY): Define.
> (machine_function): Add frame_size member.
> (THUMB_INITIAL_ELIMINATION_OFFSET): Use thumb_get_frame_size.
>
> 2002-11-05 Richard Earnshaw (rearnsha@arm.com)
>
> * arm.c (bit_count): Make argument unsigned long. Return unsigned.
> Adjust code to use portable unsigned bit manipulation.
> (insn_flags, tune_flags): Change type to unsigned.
> (struct processors): Make flags unsigned long.
> (arm_override_options): Change type of count and current_bit_count
> to unsigned.
Normally it would be too late for a patch of this complexity to be placed
on the trunk, but since this doesn't change the behaviour for existing
configurations and since we need it before we can add a netbsd/elf
configuration, then yes, this is ok for the trunk.
R.