Graphite header order

[David Edelsohn]

On Fri, Nov 20, 2015 at 11:31 AM, Sebastian Pop <sebpop@gmail.com> wrote:
> On Fri, Nov 20, 2015 at 10:23 AM, Sven Verdoolaege <skimo-isl@kotnet.org> wrote:
>> On Fri, Nov 20, 2015 at 10:14:47AM -0600, Sebastian Pop wrote:
>>> On Fri, Nov 20, 2015 at 9:31 AM, David Edelsohn <dje.gcc@gmail.com> wrote:
>>> > Sebastian,
>>> >
>>> > I have tried to build GCC with Graphite and ISL on AIX and encountered
>>> > two problems:
>>> >
>>> > (1) isl/ctx.h
>>> >
>>> > typedef enum {
>>> >         isl_stat_error = -1,
>>> >         isl_stat_ok = 0,
>>> > } isl_stat;
>>> >
>>> > GCC complains about the comma in "isl_stat_ok = 0,".  This seems like
>>> > a general bug that should appear on all targets.
>>>
>>> Sven, is there a way to fix this problem in ISL 0.15?
>>
>> Which version of gcc complains about the comma and why does it complain?
>> AFAIU, the final comma is perfectly valid in C99.
>
> On my system, gcc is compiled by g++ -std=gnu++98, then bootstrap uses
> -std=gnu++11.

Sven,

It's a C++ warning that is treated as an error during the GCC build.
This presumably is a problem on all configurations of GCC.

In file included from
/gsa/yktgsa/home/e/d/edelsohn/install/include/isl/list.h:13:0,
                 from
/gsa/yktgsa/home/e/d/edelsohn/install/include/isl/aff_type.h:4,
                 from
/gsa/yktgsa/home/e/d/edelsohn/install/include/isl/local_space.h:4,
                 from
/gsa/yktgsa/home/e/d/edelsohn/install/include/isl/constraint.h:13,
                 from /nasfarm/edelsohn/src/src/gcc/graphite.c:36:
/gsa/yktgsa/home/e/d/edelsohn/install/include/isl/ctx.h:83:17:
warning: comma at end of enumerator list [-Wpedantic]
  isl_stat_ok = 0,
                 ^

Thanks, David