Re: Patch: Add a stop bit in alignment for alloc.

["H. J. Lu" <hjl at lucon dot org>]

On Tue, Mar 02, 2004 at 11:40:23PM -0800, H. J. Lu wrote:
> On Tue, Mar 02, 2004 at 10:12:16PM -0800, David Mosberger wrote:
> > >>>>> On Tue, 2 Mar 2004 20:41:29 -0800, "H. J. Lu" <hjl@lucon.org> said:
> > 
> >   HJ> I checked in the patch. But it is not complete. If the alignment
> >   HJ> directive is a no-op, the stop bit will be missing.
> > 
> > Yeah, I guess that's counter-intuitive given that gas would normally
> > insert a stop-bit in front of the "alloc".  Having said that, putting
> > an explicit stop-bit in front of the .align seems to do the right
> > thing so hand-crafted assembly would probably be OK.  Also, for the
> > first time, the Linux kernel built and worked with -falign-loops=32!
> > Very nice!
> > 
> 
> We only want to set align_frag for ".align N". If we can look back
> a bundle, we can add a stop bit when needed. This patch seems to
> work OK. When we can't look back and we need to, it will abort.
> 

Here is the updated version.


H.J.

Attachment: gas-ia64-noop-2.patch
Description: Text document