Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-09 Thread Sven Verdoolaege
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

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-08 Thread Tobias Grosser
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 `getSch

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-08 Thread Roman Gareev
> 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

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-06 Thread Tobias Burnus
Richard Biener wrote: On Thu, Nov 6, 2014 at 8:02 AM, Tobias Grosser wrote: Also, as Tobias suggested we should raise the minimal supported isl level to 0.14 to be sure PR 62289 is fixed. As I am testing with system isl/cloog that would be unfortunate as it means I'd either drop testing graphi

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-06 Thread Tobias Grosser
On 06.11.2014 20:08, Roman Gareev wrote: As the ISL code generator has been default since a while and we did not get many bug reports, the actual switch seems to have worked well. We could probably still need some testing, but in this case it is most likely time to drop the CLooG support entirely

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-06 Thread Roman Gareev
> As the ISL code generator has been default since a while and we did not get > many bug reports, the actual switch seems to have worked well. We could > probably still need some testing, but in this case it is most likely time to > drop the CLooG support entirely. Are you interested to provide the

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-06 Thread Sven Verdoolaege
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, y

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-06 Thread Richard Biener
On 11/6/14, Tobias Grosser wrote: > On 06.11.2014 11:15, Richard Biener wrote: >> On 11/6/14, Tobias Grosser wrote: >>> On 06.11.2014 10:05, Richard Biener wrote: On Thu, Nov 6, 2014 at 8:02 AM, Tobias Grosser wrote: > On 06.11.2014 07:04, Roman Gareev wrote: >>> >>> CLooG

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-06 Thread Tobias Grosser
On 06.11.2014 11:15, Richard Biener wrote: On 11/6/14, Tobias Grosser wrote: On 06.11.2014 10:05, Richard Biener wrote: On Thu, Nov 6, 2014 at 8:02 AM, Tobias Grosser wrote: On 06.11.2014 07:04, Roman Gareev wrote: CLooG is not necessarily needed. You can run graphite just with ISL. The ma

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-06 Thread Richard Biener
On 11/6/14, Tobias Grosser wrote: > On 06.11.2014 10:05, Richard Biener wrote: >> On Thu, Nov 6, 2014 at 8:02 AM, Tobias Grosser wrote: >>> On 06.11.2014 07:04, Roman Gareev wrote: > > CLooG is not necessarily needed. You can run graphite just with ISL. > The > main reason that IS

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-06 Thread Tobias Grosser
On 06.11.2014 10:05, Richard Biener wrote: On Thu, Nov 6, 2014 at 8:02 AM, Tobias Grosser wrote: On 06.11.2014 07:04, Roman Gareev wrote: CLooG is not necessarily needed. You can run graphite just with ISL. The main reason that ISL code generation is not enabled by default is that we did not

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-06 Thread Richard Biener
On Thu, Nov 6, 2014 at 8:02 AM, Tobias Grosser wrote: > On 06.11.2014 07:04, Roman Gareev wrote: >>> >>> CLooG is not necessarily needed. You can run graphite just with ISL. The >>> main reason that ISL code generation is not enabled by default is that we >>> did not yet get extensive testing and

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-05 Thread Tobias Grosser
On 06.11.2014 07:04, Roman Gareev wrote: CLooG is not necessarily needed. You can run graphite just with ISL. The main reason that ISL code generation is not enabled by default is that we did not yet get extensive testing and it was unclear who will have the time to fix possible bugs. Could you

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-05 Thread Roman Gareev
> CLooG is not necessarily needed. You can run graphite just with ISL. The > main reason that ISL code generation is not enabled by default is that we > did not yet get extensive testing and it was unclear who will have the time > to fix possible bugs. Could you please advise me which test suites

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-05 Thread Tobias Grosser
On 04.11.2014 16:17, Tobias Burnus wrote: Hi all, currently, contrib/download_prerequisites downloads isl-0.12.2 from ftp://gcc.gnu.org/pub/gcc/infrastructure/$ISL.tar.bz2 However, that version has a bug which causes an ICE (PR 62289). End of October, ISL 0.14 was released, which should contain

Re: RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-04 Thread Richard Biener
On Tue, Nov 4, 2014 at 4:17 PM, Tobias Burnus wrote: > Hi all, > > currently, contrib/download_prerequisites downloads isl-0.12.2 from > ftp://gcc.gnu.org/pub/gcc/infrastructure/$ISL.tar.bz2 > > However, that version has a bug which causes an ICE (PR 62289). > End of October, ISL 0.14 was released

RFC: Update ISL under gcc/infrastructure/ ? // Remove CLooG?

2014-11-04 Thread Tobias Burnus
Hi all, currently, contrib/download_prerequisites downloads isl-0.12.2 from ftp://gcc.gnu.org/pub/gcc/infrastructure/$ISL.tar.bz2 However, that version has a bug which causes an ICE (PR 62289). End of October, ISL 0.14 was released, which should contain a fix for that issue. Hence, one should con