Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-23 Thread Richard Sandiford
"Moore, Catherine" writes: > 2013-03-21 Catherine Moore > > * config/mips/constraints.md (u, Udb7 Uead, Uean, Uesp, Uib3, > Uuw6, Usb4, ZS, ZT, ZU, ZV, ZW): New constraints. > * config/mip/predicates.md (lwsp_swsp_operand, > lw16_sw16_operand, lhu16_sh16_operand, lbu16_

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-22 Thread Moore, Catherine
> -Original Message- > From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > Sent: Thursday, March 21, 2013 8:04 PM > To: Moore, Catherine > Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej > Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support > > Thanks,

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-21 Thread Richard Sandiford
Thanks, this is almost there now. It was only the problem with the new version of the move pattern (see below) that stopped this from being "OK with...". The next round should be a formality though. "Moore, Catherine" writes: > +(define_constraint "Uuw6" > + "@internal > + An unsigned consta

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-21 Thread Moore, Catherine
> -Original Message- > From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > Sent: Thursday, March 21, 2013 4:05 AM > To: Moore, Catherine > Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej > Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support > > &q

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-21 Thread Richard Sandiford
"Moore, Catherine" writes: > I'm sorry for wasting your time. I accidentally posted an older version > of the patch earlier this afternoon. > This is the version that I meant to post and is hopefully a lot closer > to what you are looking for. > I named some of the predicates/constraints differen

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-20 Thread Moore, Catherine
rrent suggestions, but hopefully they will be OK. Thanks, Catherine > -Original Message- > From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > Sent: Wednesday, March 20, 2013 5:48 PM > To: Moore, Catherine > Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej > Subject

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-20 Thread Richard Sandiford
"Moore, Catherine" writes: > Index: gcc/config/mips/constraints.md > === > --- gcc/config/mips/constraints.md(revision 196638) > +++ gcc/config/mips/constraints.md(working copy) > @@ -43,6 +43,9 @@ > (define_register_constrai

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-20 Thread Moore, Catherine
> -Original Message- > From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > Sent: Tuesday, March 19, 2013 4:38 PM > To: Moore, Catherine > Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej > Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support > > &q

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-19 Thread Moore, Catherine
> -Original Message- > From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > Sent: Tuesday, March 19, 2013 3:26 PM > To: Moore, Catherine > Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej > Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support > > &q

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-19 Thread Richard Sandiford
"Moore, Catherine" writes: >> -Original Message- >> From: Richard Sandiford [mailto:rdsandif...@googlemail.com] >> Sent: Tuesday, March 19, 2013 3:26 PM >> To: Moore, Catherine >> Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej >> Subject

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-19 Thread Richard Sandiford
"Moore, Catherine" writes: >> >> -Original Message- >> >> From: Richard Sandiford [mailto:rdsandif...@googlemail.com] >> >> Sent: Tuesday, March 05, 2013 4:06 PM >> >> To: Moore, Catherine >> >> Cc: gcc-patches@gcc.gn

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-15 Thread Moore, Catherine
Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej > Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support > > "Moore, Catherine" writes: > >> -Original Message- > >> From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > >> Sent: Tuesday, March 05

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-14 Thread Richard Sandiford
"Moore, Catherine" writes: >> -Original Message- >> From: Richard Sandiford [mailto:rdsandif...@googlemail.com] >> Sent: Tuesday, March 05, 2013 4:06 PM >> To: Moore, Catherine >> Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej >> Subject

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-13 Thread Moore, Catherine
> -Original Message- > From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > Sent: Tuesday, March 05, 2013 4:06 PM > To: Moore, Catherine > Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej > Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support: > > We

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-05 Thread Richard Sandiford
"Moore, Catherine" writes: >> -Original Message- >> From: Richard Sandiford [mailto:rdsandif...@googlemail.com] >> Sent: Monday, March 04, 2013 3:54 PM >> To: Moore, Catherine >> Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej >> Subject

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-04 Thread Moore, Catherine
> -Original Message- > From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > Sent: Monday, March 04, 2013 3:54 PM > To: Moore, Catherine > Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej > Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support > > "Moor

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-04 Thread Richard Sandiford
"Moore, Catherine" writes: > Hi Richard, > I've attached an example patch for the add pattern that tries to > identify the short microMIPS variants. In your last review, you > mentioned that you would like to see the register requirements modeled > in the patterns. Do you have any comments on th

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-03-04 Thread Moore, Catherine
Hi Richard, I've attached an example patch for the add pattern that tries to identify the short microMIPS variants. In your last review, you mentioned that you would like to see the register requirements modeled in the patterns. Do you have any comments on the approach that I'm taking? Thanks

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-02-25 Thread Moore, Catherine
> -Original Message- > From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > Sent: Monday, February 25, 2013 4:42 AM > To: Moore, Catherine > Cc: gcc-patches@gcc.gnu.org; Rozycki, Maciej > Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support > > "Moor

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-02-25 Thread Richard Sandiford
"Moore, Catherine" writes: > HI Richard, > The base patch is now committed. The final patch, including your final > edits is attached. I will be posting the optimization pieces later this > week. Sorry, when I said that it was ok for 4.9, I meant that it should wait until 4.8 had branched. As

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-02-21 Thread Richard Sandiford
"Moore, Catherine" writes: >> Looks good otherwise, but please post an updated patch. It's probably >> stating the obvious, but the patch is too invasive for this late stage >> of 4.8, so >> it'll need to wait until 4.9. >> > > The updated patch is attached. How's it looking this time? Almost

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-02-19 Thread Maciej W. Rozycki
On Tue, 19 Feb 2013, Richard Sandiford wrote: > > Index: gcc.target/mips/umips-lwp-swp-2.c > > === > > --- gcc.target/mips/umips-lwp-swp-2.c (revision 0) > > +++ gcc.target/mips/umips-lwp-swp-2.c (revision 0) > > @@ -0,0 +

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-02-19 Thread Richard Sandiford
Thanks, this is looking much better now. "Moore, Catherine" writes: > I'd also like to support -mno-jals for backward compatibility. Are you > okay with that? If so, I'll submit as a separate patch. Hmm, to be honest, I'd rather you kept it local to the Mentor toolchain. The way the patch is w

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-02-13 Thread Richard Sandiford
"Moore, Catherine" writes: >> > Index: config/mips/mips.c >> > >> == >> = >> > --- config/mips/mips.c (revision 195351) >> > +++ config/mips/mips.c (working copy) >> > @@ -77,6 +77,9 @@ along with GCC; see the file COPYING3.

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-01-28 Thread Richard Sandiford
"Maciej W. Rozycki" writes: > On Sat, 26 Jan 2013, Richard Sandiford wrote: > >> > How about instead of complicating this we simply add support for >> > microMIPS encoding in the PLT? I think I should be able to squeeze out >> > some time next week to dust off and retest the binutils patch I'v

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-01-27 Thread Maciej W. Rozycki
On Sat, 26 Jan 2013, Richard Sandiford wrote: > > How about instead of complicating this we simply add support for > > microMIPS encoding in the PLT? I think I should be able to squeeze out > > some time next week to dust off and retest the binutils patch I've had > > pending far too long now

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-01-26 Thread Richard Sandiford
"Maciej W. Rozycki" writes: > On Wed, 23 Jan 2013, Richard Sandiford wrote: >> > Index: config/mips/gnu-user.h >> > === >> > --- config/mips/gnu-user.h (revision 195351) >> > +++ config/mips/gnu-user.h (working copy) >> > @@ -137,3 +1

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-01-25 Thread Maciej W. Rozycki
On Wed, 23 Jan 2013, Richard Sandiford wrote: > > Index: config/mips/gnu-user.h > > === > > --- config/mips/gnu-user.h (revision 195351) > > +++ config/mips/gnu-user.h (working copy) > > @@ -137,3 +137,12 @@ extern const char *host_

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2013-01-23 Thread Richard Sandiford
Hi Catherine, Thanks for the update. Despite the length of this reply, there isn't anything major. Please don't change all NOMIPS16 to NOCOMPRESSION in the testsuite. Most NOMIPSs are there because of the drastically reduced MIPS16 instruction set. I would have expected most of them to pass wit

RE: FW: [PATCH] [MIPS] microMIPS gcc support

2013-01-22 Thread Moore, Catherine
rine > Cc: gcc-patches@gcc.gnu.org > Subject: Re: FW: [PATCH] [MIPS] microMIPS gcc support > > "Moore, Catherine" writes: > > Forgot to copy the list ... > > Same with my reply... > > > Here is the updated microMIPS patch. It's been a very long time

Re: FW: [PATCH] [MIPS] microMIPS gcc support

2012-07-19 Thread Richard Sandiford
"Moore, Catherine" writes: > Forgot to copy the list ... Same with my reply... > Here is the updated microMIPS patch. It's been a very long time (two > years!) since I posted the original. Please let me know what we're > going to need to do to get this committed. This is looking better. At l