This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Merging Graphite to trunk
On Fri, Jul 31, 2009 at 07:56, Jack Howarth<howarth@bromo.med.uc.edu> wrote:
> On Thu, Jul 30, 2009 at 11:58:55PM -0500, Sebastian Pop wrote:
>> On Thu, Jul 30, 2009 at 22:59, Jack Howarth<howarth@bromo.med.uc.edu> wrote:
>> > Sebastian,
>> > Â On x86_64-apple-darwin10, gcc trunk is failing to bootstrap
>> > at r150304 with the failure...
>> [...]
>> > /sw/include/cloog/ppl_backend.h:178:6: error: using 'polyhedron' as both a typedef and a tag is invalid in C++
>>
>> This has been fixed by Ian in CLooG-PPL. ÂThe minimal version required
>> to bootstrap GCC4.5 is CLooG-PPL-0.15.4. ÂPlease update to
>> ftp://gcc.gnu.org/pub/gcc/infrastructure/cloog-ppl-0.15.4.tar.gz
>>
>> Sebastian
>
> Sebastian,
> Â Shouldn't configure be patched to test for cloog >= 0.15.4 in gcc trunk?
Yes, this is correct, we should test for the revision number as well.
Here is the patch, but that would necessitate a new CLooG-PPL revision.
diff --git a/configure.ac b/configure.ac
index 22cd976..578721a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1578,7 +1578,7 @@ if test "${ENABLE_CLOOG_CHECK}" = "yes"; then
CFLAGS="$CFLAGS $clooginc $gmpinc $pplinc"
AC_MSG_CHECKING([for correct version of CLooG])
AC_TRY_COMPILE([#include "cloog/cloog.h"],[
- #if CLOOG_VERSION_MAJOR != 0 || CLOOG_VERSION_MINOR != 15
+ #if CLOOG_VERSION_MAJOR != 0 || CLOOG_VERSION_MINOR != 15 ||
CLOOG_VERSION_REVISION < 5
choke me
#endif
], [AC_MSG_RESULT([yes])], [AC_MSG_RESULT([no]); clooglibs= ; clooginc= ])
Sebastian