Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Alan Modra <amodra@bigpond.net.au>
To: Andrew Cagney <ac131313@cygnus.com>
Cc: binutils@sources.redhat.com, gdb@sources.redhat.com
Subject: Re: objdump -M for x86
Date: Sat, 03 Nov 2001 01:01:00 -0000	[thread overview]
Message-ID: <20011114200754.M6922@bubble.sa.bigpond.net.au> (raw)
In-Reply-To: <3BF23030.4070201@cygnus.com>; from ac131313@cygnus.com on Wed, Nov 14, 2001 at 03:49:52AM -0500

On Wed, Nov 14, 2001 at 03:49:52AM -0500, Andrew Cagney wrote:
> 
> >  mode_64bit = (info->mach == bfd_mach_x86_64_intel_syntax
> >  		|| info->mach == bfd_mach_x86_64);
> >  
> > +  if (intel_syntax == -1)
> > +    intel_syntax = (info->mach == bfd_mach_i386_i386_intel_syntax
> > +		    || info->mach == bfd_mach_x86_64_intel_syntax);
> > +
> 
> See my other e-mail.  I can't find anything that sets ->mach to 
> .._intel_syntax so, apart from backward compatibility, I can't think of 
> a reason to hang on to those ``machine'' variants.  I don't expect GDB 
> to start using them again.

"objdump -d -m i386:intel" sets it via bfd_default_scan from the entry
in bfd/cpu-i386.c

> We'll need to do things like pin down the semantics.  Which really means 
> agree on what something like ``x86-64 i386 i8086 intel att addr16 addr32 
> data32 data16 suffix'' actually means, and ensure that GDB doesn't find 
> it has accepted a set of options only to find them later rejected by the 
> disasssembler.

Perhaps the thing to do is add a function to bfd_arch_info_type that
validates a -M string for the given arch.  If a null string is passed in,
it could return an array of possible string components.

Alan


  reply	other threads:[~2001-11-14  9:38 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20011114134429.G6922@bubble.sa.bigpond.net.au>
2001-11-02 12:10 ` Andrew Cagney
2001-11-02 15:12   ` Alan Modra
2001-11-02 23:08     ` Andrew Cagney
2001-11-03  1:01       ` Alan Modra [this message]
2001-11-27 22:37         ` Andrew Cagney
2001-11-18 12:47           ` Andrew Cagney

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=20011114200754.M6922@bubble.sa.bigpond.net.au \
    --to=amodra@bigpond.net.au \
    --cc=ac131313@cygnus.com \
    --cc=binutils@sources.redhat.com \
    --cc=gdb@sources.redhat.com \
    /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