This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Add POWER6 machine description
- From: Roger Sayle <roger at eyesopen dot com>
- To: David Edelsohn <dje at watson dot ibm dot com>, Peter Bergner <bergner at vnet dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sat, 28 Oct 2006 19:26:50 -0600 (MDT)
- Subject: Re: [PATCH] Add POWER6 machine description
Hi Peter and David,
On Fri, 27 Oct 2006, David Edelsohn wrote:
> The recog.c change needs to be posted separately and reviewed
> separately. It is an independent change and imposes a dependency on the
> rest of the patch. It needs to be approved first.
>
>> * recog.c (store_data_bypass_p): Add support to allow IN_INSN to
>> be a PARALLEL containing sets. Return false when out_pat is not
>> a PARALLEL insn.
The recog.c change looks good. As this is just a generalization of
the current store_data_bypass_p, where we'd currently gcc_assert if
not passed a single set, it shouldn't affect any other platform.
This part is OK for mainline. I'll let you decide whether you want
to commit this separately (my preference), or at the same time as
the revised power6 patch, once that's approved.
Roger
--