public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: Jim Wilson <wilson@specifixinc.com>
To: linux-ia64@vger.kernel.org
Subject: Re: [PATCH] new ".serialize" gas directive
Date: Wed, 16 Jun 2004 22:42:40 +0000	[thread overview]
Message-ID: <1087425760.1178.82.camel@leaf.tuliptree.org> (raw)
In-Reply-To: <16537.51724.854691.934006@napali.hpl.hp.com>

On Wed, 2004-06-16 at 15:04, Chris Wedgwood wrote:
> 'Current' is what these days? :)

Well, I used something from the binutils cvs tree to check this, but you
may not want to do that.  Whatever is HJ's most recent binutils release
is probably what you should try using.

> Yeah, I guess that's true so in this case I'm not sure I can blame gas
> entirely, however the scope is limited and it's not actually used
> outside of the macro so gas should be able to infer something about
> the program flow.

Tags can't be used for control flow, so that should never have been an
issue.  I believe the problem here is that a label was used before gas
had tag support, and the pred.rel was required at that time.  The
.pred.rel could have been deleted when the label was converted to a tag,
but this was missed in the conversion.

If there is a label here, then there is nothing gas can do except assume
the worst, as gas has no support for building a control flow graph.  Nor
is there any easy place to put it.  Gas is a simple two pass assembler,
one pass to read the input, and a second pass to back patch stuff that
could not be calculated during the first pass.  It isn't possible to
construct a control flow graph until after the first pass, and by then
it is too late, as we have already done all of the dependency violation
analysis during the first pass.  This could be fixed, but it would
require a major rewrite of the IA-64 gas port, and it isn't clear that
this is worth the trouble.
-- 
Jim Wilson, GNU Tools Support, http://www.SpecifixInc.com


  parent reply	other threads:[~2004-06-16 22:42 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-05-06  5:15 [PATCH] new ".serialize" gas directive David Mosberger
2004-05-06 23:33 ` Jim Wilson
2004-05-06 23:47 ` David Mosberger
2004-05-07  7:48 ` Jim Wilson
2004-05-10 20:16 ` David Mosberger
2004-05-14  8:43 ` Jim Wilson
2004-05-14 18:30 ` David Mosberger
2004-06-16  1:54 ` Chris Wedgwood
2004-06-16 21:57 ` Jim Wilson
2004-06-16 22:04 ` Chris Wedgwood
2004-06-16 22:42 ` Jim Wilson [this message]
2004-06-16 22:56 ` Luck, Tony
2004-06-16 23:01 ` David Mosberger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1087425760.1178.82.camel@leaf.tuliptree.org \
    --to=wilson@specifixinc.com \
    --cc=linux-ia64@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox