Re: Graphite header order
On Fri, Nov 20, 2015 at 10:14:47AM -0600, Sebastian Pop wrote: > On Fri, Nov 20, 2015 at 9:31 AM, David Edelsohn 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. skimo
Re: [GSoC] questions about graphite_clast_to_gimple.c
On Tue, May 06, 2014 at 01:02:09PM +0200, Tobias Grosser wrote: > I am busy for the next 6 weeks, but am planning to work on the isl patch > after. Sven, do you happen to have any time to work on the isl patch? No. It's going to be difficult enough for me to finish what I need to finish for the CARP project. I may have more time in July. Then again, I may also have no time at all if I need to go and find a "real job". skimo
Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?
On Thu, Nov 06, 2014 at 10:43:16AM +0100, Tobias Grosser wrote: > is there any chance we can add the deprecated isl_int includes back into isl > 0.14.1. This would unblock the testing and we could remove them as soon as > gcc 4.8/4.9 has been phased out. I prefer Richard's solution. In any case, you need to test later versions of isl before you accept for use in older versions of gcc. Even if there are no incompatibilities in the API, the internal representations of results of operations may very well change, which may, for example, result in significantly different output from the AST generator. skimo
Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?
On Sat, Nov 08, 2014 at 09:01:14PM +0100, Tobias Grosser wrote: > On 08.11.2014 20:49, Roman Gareev wrote: > >>Sounds good as long as they will compile and pass tests independently. > >>Please also remember updating the documentation. > > > >I’m trying to build Graphite without CLooG, but I get the following error: > > > >libbackend.a(graphite-optimize-isl.o): In function `getScheduleForBandList': > >/home/roman/sec_trunk/gcc/gcc/graphite-optimize-isl.c:357: undefined > >reference to `isl_band_member_is_zero_distance' > > @Sven: Maybe Sven has an idea how to handle this best. >From the manual: =item * The function C has been removed. Essentially the same functionality is available through C, except that is requires setting up coincidence constraints. skimo