This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch AArch64] Do not increase data alignment at -Os and with -fconserve-stack.
- From: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- To: Ramana Radhakrishnan <ramana dot radhakrishnan at foss dot arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, James Greenhalgh <james dot greenhalgh at arm dot com>
- Date: Tue, 6 Jun 2017 14:34:39 +0100
- Subject: Re: [Patch AArch64] Do not increase data alignment at -Os and with -fconserve-stack.
- Authentication-results: sourceware.org; auth=none
- References: <3f42e722-36e6-ad2b-29ec-84aab588cafe@foss.arm.com>
Ping..
Ramana
On Tue, May 2, 2017 at 10:52 AM, Ramana Radhakrishnan
<ramana.radhakrishnan@foss.arm.com> wrote:
> We unnecessarily align data to 8 byte alignments even when -Os is specified.
> This brings the logic in the AArch64 backend more in line with the ARM
> backend and helps gain some image size in a few places. Caught by an
> internal report on the size of rodata sections being high with aarch64 gcc.
>
> * config/aarch64/aarch64.h (AARCH64_EXPAND_ALIGNMENT): New.
> (DATA_ALIGNMENT): Update to use AARCH64_EXPAND_ALIGNMENT.
> (LOCAL_ALIGNMENT): Update to use AARCH64_EXPAND_ALIGNMENT.
>
> Bootstrapped and regression tested on aarch64-none-linux-gnu with no
> regressions.
>
> Ok to commit ?
>
>
> cheers
> Ramana
>