Re: [PATCH 0/5] Remove the LAMBDA framework

[Jack Howarth <howarth@bromo.med.uc.edu>]

On Mon, Jan 17, 2011 at 11:37:58PM -0600, Sebastian Pop wrote:
> Hi Richi,
> 
> Here is the second part of the patches that completely remove the
> LAMBDA framework and fixes at least the following PRs from my todo
> list: 29832, 43567, 43657, 46168.
> 
> The patches remove the XFAIL from the loop kernel interchange-3.f90
> that represents the swim kernel.  The ltrans-* testcases are now in
> the graphite testsuite and all passing except interchange-2.c that has
> a SCoP detection problem: a loop with two exits is not yet handled in
> the Graphite framework.  This will be fixed later.
> 
> I will commit these patches to the graphite branch for further tests
> sometime tomorrow if there are no further comments.

Sebastian,
   Are there duplications in the fifth patch?

[PATCH 5/5] Remove the lambda framework and make -ftree-loop-linear an alias of -floop-interchange.

When I apply the 10 patches from the series in http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01036.html,
the follow patch from http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01117.html and these five patches,
the one from http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01186.html gives...

patching file gcc/lambda-mat.c
Reversed (or previously applied) patch detected!  Assume -R? [n] 
Apply anyway? [n] 
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file gcc/lambda-mat.c.rej
patching file gcc/lambda-trans.c
Reversed (or previously applied) patch detected!  Assume -R? [n] 
Apply anyway? [n] 
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file gcc/lambda-trans.c.rej
patching file gcc/lambda.h
Reversed (or previously applied) patch detected!  Assume -R? [n] 
Apply anyway? [n] 
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file gcc/lambda.h.rej

Thanks in advance for any clarifications.
         Jack


> 
> Thanks,
> Sebastian
> 
> Sebastian Pop (5):
>   Relax the definition of same_pdr_p.
>   Remove duplicate close phi nodes in the canonical close phi
>     representation.
>   Allow several reductions in a reduction PBB.
>   Do not create the temporary array for reductions into VAR_DECLs.
>   Remove the lambda framework and make -ftree-loop-linear an alias of
>     -floop-interchange.
> 
>  ChangeLog.graphite                                 |    4 +
>  MAINTAINERS                                        |    1 -
>  gcc/ChangeLog.graphite                             |   86 +
>  gcc/Makefile.in                                    |   17 +-
>  gcc/graphite-dependences.c                         |   10 +-
>  gcc/graphite-poly.h                                |    3 +-
>  gcc/graphite-scop-detection.c                      |   69 +-
>  gcc/graphite-sese-to-poly.c                        |    7 +-
>  gcc/lambda-code.c                                  | 2855 --------------------
>  gcc/lambda-mat.c                                   |  607 -----
>  gcc/lambda-trans.c                                 |   80 -
>  gcc/lambda.h                                       |  524 ----
>  gcc/lto-symtab.c                                   |    1 -
>  gcc/passes.c                                       |    1 -
>  .../gfortran.dg/graphite/interchange-3.f90         |    2 +-
>  gcc/tree-data-ref.c                                |  174 ++
>  gcc/tree-data-ref.h                                |  143 +-
>  gcc/tree-loop-linear.c                             |  423 ---
>  gcc/tree-parloops.c                                |  119 +
>  gcc/tree-pass.h                                    |    3 +-
>  gcc/tree-ssa-loop.c                                |   44 +-
>  21 files changed, 603 insertions(+), 4570 deletions(-)
>  delete mode 100644 gcc/lambda-code.c
>  delete mode 100644 gcc/lambda-mat.c
>  delete mode 100644 gcc/lambda-trans.c
>  delete mode 100644 gcc/lambda.h
>  delete mode 100644 gcc/tree-loop-linear.c