This is the mail archive of the 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 0/3] Configure support for official CLooG versions.

On Wed, Aug 11, 2010 at 08:30:09PM +0200, Andreas Simbürger wrote:
> Hi,
> this patch series adds configure support for upstream CLooG versions.
> The official CLooG is able to use three different backends during
> code generation. An ISL backend, a PPL backend and a PolyLib backend.
> Furthermore the old CLooG-PPL (Legacy) fork used by graphite at the
> moment is still supported.
> While checking for the used CLooG version we check in the following
> order: PPL (Legacy) > ISL > PPL > PolyLib.

One issue that remains to be resolved is a mechanism to retain
coherency between the gcc and cloog builds. Currently one can
end up with the following situation...

1) Build and install cloog-ppl against ppl-0.10.2.
2) Build and install FSF gcc against cloog-ppl and ppl-10.2.
3) Build and install ppl-0.11.
4) Rebuild and install cloog-ppl against ppl-0.11.

This creates an ABI mismatch since the calls to cloog
in gcc are compiled using the ppl-0.10.2 headers (as
loaded via cloog.h) while the called subroutine in
the installed cloog was actually built against a
newer ABI from ppl.0.11.
   There should be a cloog call added to allow the
calling program to confirm that the version of isl,
ppl or polylib that it was built against is of the
same ABI as that for the isl, ppl or polylib that
cloog itself was built against.

> This means, we don't change the default behavior of CLooG yet.
> As we still need to discuss some aspects of providing information 
> with upstream it is necessary to disable the cloog version check
> (--disable-cloog-version-check) to use one of the upstream
> CLooG versions for now.
> After successfully completing all test suites we want to switch
> to CLooG-ISL and abandon the legacy CLooG-PPL.
> Andreas
> Andreas Simbürger (3):
>   Add necessary accessors for CloogMatrix (CLOOG_ORG).
>   Fix include order to avoid failed compilation on CLOOG_ORG.
>   Support official versions.
>  configure                   |  156 +++++++++++++++++++++++++++++++++++++------
>                |   90 ++++++++++++++++++------
>  gcc/ChangeLog.graphite      |   14 ++++
>  gcc/graphite-cloog-compat.h |   10 +++
>  gcc/graphite-poly.c         |    2 +-
>  5 files changed, 228 insertions(+), 44 deletions(-)
> -- 
> 1.7.2

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