This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] s390: z10 scheduler fixes part 3/4
- From: Wolfgang Gellerich <gellerich at de dot ibm dot com>
- To: Andreas Krebbel <krebbel at linux dot vnet dot ibm dot com>, Ulrich Weigand <Ulrich dot Weigand at de dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 29 Jan 2009 08:32:37 +0100
- Subject: Re: [PATCH] s390: z10 scheduler fixes part 3/4
- References: <49807D2D.6070905@linux.vnet.ibm.com> <49808F28.8020109@linux.vnet.ibm.com>
> Andreas Krebbel <krebbel@linux.vnet.ibm.com>
> 01/28/2009 06:00 PM
>
> To
>
> Wolfgang Gellerich/Germany/IBM@IBMDE, gcc-patches@gcc.gnu.org
>
> cc
>
> Subject
>
> Re: [PATCH] s390: z10 scheduler fixes part 3/4
>
> Wolfgang Gellerich wrote:
> > The third patch provides a correct declaration of the z10 pipeline
> > properties for instructions iihf and iilf, respectively. Both
> > instructions were generated by an insn named *insv<mode>_reg_extimm.
The
> > easiest way to achieve this was splitting the insn into two separate
> > insns which also simplifies a condition. Bootstraps and does not
> > introduce new test case failures.
>
> Mmmh not sure about that one. I would prefer to keep it as a single
> pattern. Let me think about it.
>
I discussed that one with Uli some time ago (you were out of office at that
time, I think) and we agreed that in addition to fixing the z10prop
attribute, it is a win to get rid of this somewhat weird condition.
Regards, Wolfgang