This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [m68k 09/13] Convert some text peepholes
On Tue, 2007-02-06 at 17:18 +0000, Richard Sandiford wrote:
> Jeffrey Law <law@redhat.com> writes:
> > On Tue, 2007-02-06 at 07:38 +0000, Richard Sandiford wrote:
> >> It's your call, but seeing as the patch I posted has been approved,
> >> and is only waiting on an unapproved dependent patch, it might be
> >> better to hold off this discussion until either (a) that patch gets
> >> reviewed, or (b) the approval of the patch I posted is reversed.
> >> (It's only a suggestion -- doing patches against mainline is obviously
> >> the right thing to do, and I'm not trying to imply otherwise.
> >> Also, if there's something you object to in that patch -- the UNSPEC? --
> >> then I can try to fix it.)
> > Which pending patch is your peephole conversion patch dependent upon?
>
> Well, it was just a conflict: it only applies cleanly on top of the
> rtl prologue/epilogue patch, which in turn depends on:
>
> [ColdFire 27/63] Addressing mode changes and fixes
> http://gcc.gnu.org/ml/gcc-patches/2007-01/msg00784.html
>
> which in turn depends on:
>
> [ColdFire 25/63] Define MODE_INDEX_REG_CLASS for m68k
> http://gcc.gnu.org/ml/gcc-patches/2007-01/msg00782.html
Ah. I'm still not comfortable with the MOD_INDEX_REG_CLASS
patch. Is there any way to break out the peephole patch
separately so that we can then go forward with Roman's
peephole patches?
jeff