This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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: Enable 32-bit Alignment


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.


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