All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Henderson <rth@twiddle.net>
To: qemu-devel@nongnu.org
Cc: blauwirbel@gmail.com
Subject: [Qemu-devel] [PATCH 0/5] tcg-sparc improvements
Date: Mon, 11 Jan 2010 11:22:06 -0800	[thread overview]
Message-ID: <cover.1263237726.git.rth@twiddle.net> (raw)

As noticed elsewhere on the list mulu2_i32 is missing.  Which led 
me to notice that add2_i32 and sub2_i32 were also missing and also
led me to notice that division was incorrectly implemented.

My initial implementation of 32-bit division copied the existing practice
of using the 64-bit division insns for sparcv9, with proper sign/zero
extensions on the input operands.  But that required extra temporaries
and quickly got ugly.  In the end I thought it was clearer to simply use
the 32-bit division insns unconditionally.

That excercise, however, made me notice that 32-bit extension was not
special-cased, leading to the final patch.


r~


Richard Henderson (5):
  tcg-sparc: Add tcg_out_arithc.
  tcg-sparc: Implement add2, sub2, mulu2.
  tcg-sparc: Do not remove %o[012] from 'r' constraint.
  tcg-sparc: Implement division properly.
  tcg-sparc: Implement ext32[su]_i64

 tcg/sparc/tcg-target.c |  206 ++++++++++++++++++++++++++++++++----------------
 tcg/sparc/tcg-target.h |    8 ++
 2 files changed, 145 insertions(+), 69 deletions(-)

             reply	other threads:[~2010-01-11 19:49 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-11 19:22 Richard Henderson [this message]
2010-01-11 17:51 ` [Qemu-devel] [PATCH 1/5] tcg-sparc: Add tcg_out_arithc Richard Henderson
2010-01-11 17:57 ` [Qemu-devel] [PATCH 2/5] tcg-sparc: Implement add2, sub2, mulu2 Richard Henderson
2010-01-11 19:00 ` [Qemu-devel] [PATCH 3/5] tcg-sparc: Do not remove %o[012] from 'r' constraint Richard Henderson
2010-01-11 19:09 ` [Qemu-devel] [PATCH 4/5] tcg-sparc: Implement division properly Richard Henderson
2010-01-11 20:11   ` Richard Henderson
2010-01-11 19:21 ` [Qemu-devel] [PATCH 5/5] tcg-sparc: Implement ext32[su]_i64 Richard Henderson
2010-01-11 20:41 ` [Qemu-devel] [PATCH 0/5] tcg-sparc improvements Palle Lyckegaard
2010-01-12 20:02 ` [Qemu-devel] " Blue Swirl

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=cover.1263237726.git.rth@twiddle.net \
    --to=rth@twiddle.net \
    --cc=blauwirbel@gmail.com \
    --cc=qemu-devel@nongnu.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.