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]

Enable 32-bit Alignment


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.

Thanks,

-- 
_______________________________________________________
Evandro Menezes               AMD            Austin, TX

Attachment: nop.patch
Description: nop.patch


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