This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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
--


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]