This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][modulo-sched] Remove profitability check
- From: Revital1 Eres <ERES at il dot ibm dot com>
- To: Ayal Zaks <ZAKS at il dot ibm dot com>
- Cc: abel at ispras dot ru, gcc-patches at gcc dot gnu dot org, Kenneth dot Zadeck at NaturalBridge dot com, volodyan at gmail dot com
- Date: Wed, 8 Aug 2007 09:27:16 +0300
- Subject: Re: [PATCH][modulo-sched] Remove profitability check
gcc-patches-owner@gcc.gnu.org wrote on 07/08/2007 22:55:16:
> Revital1 Eres/Haifa/IBM wrote on 07/08/2007 15:16:58:
>
> > Hello,
> >
> > This patch is the third one in the series of patches originated from
> > patch 1 of 2 (http://gcc.gnu.org/ml/gcc-patches/2007-01/msg01468.html).
> >
> > A reminder - here is the list of issues patch 1 of 2 addresses:
> >
> > 1.1 Avoid SMS when the loop contains inc instruction. (commited)
> > 1.2 Fix removal of anti-deps. (commited)
> > 1.3 Add -fsms-allow-reg-moves flag. (commited)
> > 1.4 Fix order of instructions within one cycle.
> > 1.5 Remove profitability checks.
> >
> > The attached patch addresses items 1.4 and 1.5 above.
> > It also adds dump information.
> >
> > A new reduced Fortran testcase for forall_10.f90 test (sms-2.f90) was
> > added; this testcase failed with -fmodulo-sched flag when the current
> > patch is not applied. It caused by applying wrong order of instructions
> > within one cycle which is fixed in the current patch.
> >
> > :ADDPATCH middle-end (modulo-sched):
> >
> > This patch was bootstrapped and tested on PPC and x86_64 (also with
> > --enable-checking=assert), with and without
-fmodulo-sched-allow-regmoves
> > flag; with no new regressions.
> >
> > OK for mainline?
> >
>
> OK.
> Please include your name in ChangeLog, as this is not entirely Vladimir's
> original patch, and see minor correction below.
Sure, I'll commit the patch with this change.
Thanks,
Revital