This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH RS6000] adding an unsigned SI and a DI together and subtracting an unsigned SI from an DI
- From: Andrew Pinski <pinskia at physics dot uc dot edu>
- To: David Edelsohn <dje at watson dot ibm dot com>
- Cc: Andrew Pinski <pinskia at physics dot uc dot edu>, gcc-patches at gcc dot gnu dot org, Richard Henderson <rth at redhat dot com>
- Date: Wed, 14 May 2003 22:54:27 -0400
- Subject: Re: [PATCH RS6000] adding an unsigned SI and a DI together and subtracting an unsigned SI from an DI
You are right, I thought I had bootstrapped but I must of not, woops.
I am putting Linux on a XServe to be able to regression testing because
of a bug in the Mac OS X kernel.
I had forgot to test that case when I was testing the patch, woops..
Thanks for finding it before submitting it,
On Wednesday, May 14, 2003, at 18:35 US/Eastern, David Edelsohn wrote:
Did you bootstrap or run a regression test on any of the
define_insn_and_split versions of the patch? There is nothing to
GCC from allocating the same register to DImode output as the SImode
input. For example, input in %r3 and output in %r3 (actually %r3,%r4
pair). Your original version
[(set (match_dup 2) (const_int 0))
(set (match_dup 3) (match_dup 1))]
I believe that reversing the order of the SETs is sufficient so that
input is consumed in the first insn.