On 07/21/2011 06:09 PM, Sebastian Pop wrote:
Hi,
On Thu, Jul 21, 2011 at 07:37, Rainer Orth<r...@cebitec.uni-bielefeld.de>
wrote:
As described in the PR, the use of LANGUAGE_C in CLooG breaks C++
bootstrap on IRIX 6.5. Both MIPS and Alpha C compilers predefine
LANGUAGE_C themselves, which clashes with the CLooG macro in
<cloog/pprint.h>. In a bootstrap with C++, LANGUAGE_C of course isn't
defined, so the symbol is missing.
As a hack, I had renamed that to CLOOG_LANGUAGE_C in the installed
CLooG headers, not noticing the use of LANGUAGE_C in
graphite-clast-to-gimple.c. Since this macro name is obviously far too
generic, I propose renaming it to CLOOG_LANGUAGE_C upstream.
I forwarded this question to the cloog-dev mailing list and they like
this change.
The current patch accounts for such a change in gcc and allows the
mips-sgi-irix6.5 bootstrap to continue.
What's the best way forward from here? Could one of you propose that
upstream? Is the patch below ok for mainline?
Your patch is ok for GCC, please commit.
You can post a patch for CLooG on cloog-developm...@googlegroups.com
Don't forget to update the documentation of CLooG as well.
I am about to post a patch to CLooG. I will take care of pushing it
upstream.
Tobi
