This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: Enable 32-bit Alignment
- From: "H. J. Lu" <hjl at lucon dot org>
- To: "Menezes, Evandro" <evandro dot menezes at amd dot com>
- Cc: binutils at sourceware dot org, Michael Meissner <michael dot meissner at sbosexmb1 dot amd dot com>, Christophe Harle <christophe dot harle at txexmtae dot amd dot com>
- Date: Fri, 29 Jun 2007 14:35:10 -0700
- Subject: Re: Enable 32-bit Alignment
- References: <1E52B51E700FAF42ADA0B9432F7CCCD24907E7@SAUSEXMB1.amd.com>
On Fri, Jun 29, 2007 at 03:57:01PM -0500, Menezes, Evandro wrote:
> If it's not too late in the release cycle, I'd appreciate if this patch
> enabling optimal code alignment up to 32-byte boundaries, used by GCC
> when tuning for AMD Barcelona, were applied. It also does away with the
> special NOP sequences for AMD Opteron in 64-bit code.
>
> 2007-06-29 Evandro Menezes <evandro.menezes@amd.com>
>
> * config/tc-i386.c (handle_align_code, i386_align_code): new
> function
> to help alignment > 15 bytes, enable alignment up to 31 bytes,
>
> always use variable-length NOP for K8 too.
> * config/tc-i386.h (MAX_MEM_FOR_RS_ALIGN_CODE): change limit to
> 31 bytes.
> * testsuite/gas/i386/i386.exp: replace NOP test-case and
> remove K8-specific NOP test-case.
> * testsuite/gas/i386/nops-3-i686.d: new NOP test-case.
> * testsuite/gas/i386/x86-64-nops-1-k8.d: removed.
>
Would you mind adding a few testcases to make sure that assembler
realy does what you want?
Thanks.
H.J.