Enable 32-bit Alignment

Menezes, Evandro evandro.menezes@amd.com
Fri Jun 29 20:57:00 GMT 2007


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nop.patch
Type: application/octet-stream
Size: 10305 bytes
Desc: nop.patch
URL: <https://sourceware.org/pipermail/binutils/attachments/20070629/ec7d170f/attachment.obj>


More information about the Binutils mailing list