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: [PATCH] ARM - gas: Maximum alignment size augmented to 64

From: Daniel Jacobowitz <drow at false dot org>
To: Richard Earnshaw <Richard dot Earnshaw at buzzard dot freeserve dot co dot uk>
Cc: Daniel Gutson <dgutson at codesourcery dot com>, binutils at sourceware dot org
Date: Wed, 17 Jun 2009 17:59:29 -0400
Subject: Re: [PATCH] ARM - gas: Maximum alignment size augmented to 64
References: <4A3933BF.5020402@codesourcery.com> <1245263262.25372.7.camel@barnowl.buzzard.freeserve.co.uk>

On Wed, Jun 17, 2009 at 07:27:42PM +0100, Richard Earnshaw wrote:
> This is OK as far as it goes, but why is there a limit at all?  Other
> architectures don't seem to be restricted in this way.

>From reading the code, it seems to be a misunderstanding of the
(quite confusing) frag fields.  An align frag has a fixed part and a
repeating part.  But see the loop at the end of arm_handle_align -
it's repeating the repeating part into an allocated buffer.  That's
not needed.

This has been there since 2001.

-- 
Daniel Jacobowitz
CodeSourcery

References:
- [PATCH] ARM - gas: Maximum alignment size augmented to 64
  - From: Daniel Gutson
- Re: [PATCH] ARM - gas: Maximum alignment size augmented to 64
  - From: Richard Earnshaw

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