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

From: "Menezes, Evandro" <evandro dot menezes at amd dot com>
To: binutils at sourceware dot org
Cc: "Michael Meissner" <michael dot meissner at sbosexmb1 dot amd dot com>, "Christophe Harle" <christophe dot harle at txexmtae dot amd dot com>, "Dwarak Rajagopal" <dwarak dot rajagopal at txexmtae dot amd dot com>
Date: Tue, 3 Jul 2007 19:12:55 -0500
Subject: Enable 32-bit Alignment

After iterating with HJ and Alan, here's another spin of the patch.

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, add new
one and 
	  remove K8-specific one.
	* testsuite/gas/i386/nops-3-i686.d: new NOP test-case.
	* testsuite/gas/i386/nops-4.s: idem
	* testsuite/gas/i386/nops-4-i686.d: idem
	* testsuite/gas/i386/x86-64-nops-1-k8.d: removed.

Thanks,

-- 
_______________________________________________________
Evandro Menezes               AMD            Austin, TX

Attachment: nop.patch
Description: nop.patch

Follow-Ups:
- Re: Enable 32-bit Alignment
  - From: H.J. Lu

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