From: "Maciej W. Rozycki" <macro@codesourcery.com>
To: Richard Sandiford <rdsandiford@googlemail.com>
Cc: Alan Modra <amodra@gmail.com>, <binutils@sourceware.org>
Subject: Re: [PATCH] MIPS: Opcode membership proposal
Date: Mon, 17 Jun 2013 22:30:00 -0000 [thread overview]
Message-ID: <alpine.DEB.1.10.1306172321020.16287@tp.orcam.me.uk> (raw)
In-Reply-To: <87y5a8o6oj.fsf@talisman.default>
On Mon, 17 Jun 2013, Richard Sandiford wrote:
> > This looks a bit convoluted, and frankly I'd prefer if automake supported
> > true per-object flags with no need to resort to hacks like this, but there
> > you go. The benefit would be no need to check the rules against generated
> > ones with each automake upgrade, that is less maintenance burden -- and
> > the maintenance of our autoconf scriptery has already proved tough even
> > without that.
> >
> > Do you want me to check this alternative or would you prefer to do this
> > yourself?
>
> What do you think about explicitly initialising each field after all?
> I can easily repurpose the ASE-checking script to do that.
Great! I'm fine with that, sure. While at it we could add pinfo3 too --
some microMIPS instructions (offhand: ALNV.PS ;) ) will benefit from more
accurate data dependency tracking.
> I understand the original reason for having optional fields, but the
> workaround is beginning to feel a bit convoluted. There's also more
> room for confusion than there was originally, now that we have the
> ASE field too.
Agreed. What I think was important was not to add an extra field while
rewriting the opcode table at the same time -- that would obfuscate the
change itself.
Maciej
next prev parent reply other threads:[~2013-06-17 22:30 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-18 12:55 Maciej W. Rozycki
2010-12-20 18:15 ` David Daney
2010-12-20 18:33 ` Andrew Pinski
2011-10-31 12:47 ` Maciej W. Rozycki
2011-11-17 18:45 ` Richard Sandiford
2012-08-10 11:08 ` Maciej W. Rozycki
2012-08-12 18:33 ` Richard Sandiford
2012-08-13 14:56 ` Maciej W. Rozycki
2013-06-17 11:51 ` Alan Modra
2013-06-17 15:18 ` Maciej W. Rozycki
2013-06-17 19:53 ` Richard Sandiford
2013-06-17 22:30 ` Maciej W. Rozycki [this message]
2013-06-18 0:27 ` Alan Modra
2013-06-24 14:28 ` Maciej W. Rozycki
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=alpine.DEB.1.10.1306172321020.16287@tp.orcam.me.uk \
--to=macro@codesourcery.com \
--cc=amodra@gmail.com \
--cc=binutils@sourceware.org \
--cc=rdsandiford@googlemail.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).