similar to: [LLVMdev] [polly] removing cloog dependence in the testsuite

Hi Tobi, Sebastian Pop wrote: > Another option is to disable these tests when cloog is not available, and to > write other tests that will work with -polly-codegen-isl. I think I like this way better. The attached patches move all the test dependent on Cloog to be conditionally executed to CLOOG_FOUND. I am preparing another patch that will adapt most of the current tests to work with

Hi Tobi, Sebastian Pop wrote: > Sebastian Pop wrote: > > Another option is to disable these tests when cloog is not available, and to > > write other tests that will work with -polly-codegen-isl. > > I think I like this way better. The attached patches move all the test > dependent on Cloog to be conditionally executed to CLOOG_FOUND. I am preparing > another patch

On Wed, Nov 28, 2012, at 06:01 PM, Sebastian Pop wrote: > Hi Tobi, > > Sebastian Pop wrote: > > Sebastian Pop wrote: > > > Another option is to disable these tests when cloog is not available, and to > > > write other tests that will work with -polly-codegen-isl. > > > > I think I like this way better. The attached patches move all the test > >

At 2013-09-03 00:12:56,"Tobias Grosser" <tobias at grosser.es> wrote: >On 09/02/2013 07:44 AM, Star Tan wrote: >> At 2013-09-02 16:22:28,"Tobias Grosser" <tobias at grosser.es> wrote: >> >>> On 09/01/2013 08:02 PM, Star Tan wrote: >>>> Hi all, >>>> >>>> >>>> Attached patch file to update lit

Hi, Star Tan wrote: > At 2013-09-03 00:12:56,"Tobias Grosser" <tobias at grosser.es> wrote: > > >On 09/02/2013 07:44 AM, Star Tan wrote: > >> At 2013-09-02 16:22:28,"Tobias Grosser" <tobias at grosser.es> wrote: > >> > >>> On 09/01/2013 08:02 PM, Star Tan wrote: > >>>> Hi all, > >>>> >

Hi Sebastian, thanks for your reply! At 2013-09-27 03:33:04,"Sebastian Pop" <spop at codeaurora.org> wrote:>Hi, > >Star Tan wrote: >> At 2013-09-03 00:12:56,"Tobias Grosser" <tobias at grosser.es> wrote: >>  >> >On 09/02/2013 07:44 AM, Star Tan wrote: >> >> At 2013-09-02 16:22:28,"Tobias Grosser" <tobias at

Tobias Grosser wrote: > Regarding patch one and three: We already have the directory > test/CodeGen and test/IslCodeGen. I propose to just leave the CLooG test > cases in test/CodeGen, but to run them conditionally. I only know how to disable the test of a full directory, so if you want I can disable all the CodeGen directory (and subdirectories) when Cloog is not available. Note that

Tobias Grosser wrote: > On 12/03/2012 06:07 PM, Sebastian Pop wrote: > >Tobias Grosser wrote: > >>Regarding patch one and three: We already have the directory > >>test/CodeGen and test/IslCodeGen. I propose to just leave the CLooG test > >>cases in test/CodeGen, but to run them conditionally. > > > >I only know how to disable the test of a full

On 12/03/2012 06:07 PM, Sebastian Pop wrote: > Tobias Grosser wrote: >> Regarding patch one and three: We already have the directory >> test/CodeGen and test/IslCodeGen. I propose to just leave the CLooG test >> cases in test/CodeGen, but to run them conditionally. > > I only know how to disable the test of a full directory, so if you want I can > disable all the

On 12/03/2012 07:42 PM, Sebastian Pop wrote: > Tobias Grosser wrote: >> On 12/03/2012 06:07 PM, Sebastian Pop wrote: >>> Tobias Grosser wrote: >>>> Regarding patch one and three: We already have the directory >>>> test/CodeGen and test/IslCodeGen. I propose to just leave the CLooG test >>>> cases in test/CodeGen, but to run them conditionally.

Tobias Grosser wrote: > >I can then move the remaining tests in Cloog/CodeGen and duplicate them in > >Isl/CodeGen. What do you think? > > Fine with me (You probably just need to copy the remaining tests in > the top level directory). Ok, so I moved the OpenMP, GPGPU, and MemAccess dirs to the toplevel polly/test/{OpenMP,GPGPU,MemAccess} and then the remaining testcases to

On 12/03/2012 09:03 PM, Sebastian Pop wrote: > Tobias Grosser wrote: >>> I can then move the remaining tests in Cloog/CodeGen and duplicate them in >>> Isl/CodeGen. What do you think? >> >> Fine with me (You probably just need to copy the remaining tests in >> the top level directory). > > Ok, so I moved the OpenMP, GPGPU, and MemAccess dirs to the

Tobias Grosser wrote: > On 12/03/2012 09:03 PM, Sebastian Pop wrote: > >Tobias Grosser wrote: > >>>I can then move the remaining tests in Cloog/CodeGen and duplicate them in > >>>Isl/CodeGen. What do you think? > >> > >>Fine with me (You probably just need to copy the remaining tests in > >>the top level directory). > > >

At 2013-09-02 16:22:28,"Tobias Grosser" <tobias at grosser.es> wrote: >On 09/01/2013 08:02 PM, Star Tan wrote: >> Hi all, >> >> >> Attached patch file to update lit config for Cloog. Without it, Polly always skips Cloog testings when we run "make check-polly". > >Dear Star Tan, > >thanks a lot for the patch. It looks very

On 09/02/2013 07:44 AM, Star Tan wrote: > At 2013-09-02 16:22:28,"Tobias Grosser" <tobias at grosser.es> wrote: > >> On 09/01/2013 08:02 PM, Star Tan wrote: >>> Hi all, >>> >>> >>> Attached patch file to update lit config for Cloog. Without it, Polly always skips Cloog testings when we run "make check-polly". >>

On Tue, Nov 19, 2013 at 12:07:18PM +0100, Tobias Grosser wrote: > On 11/19/2013 08:50 PM, Jack Howarth wrote: >> Tobias, >> Can we add something like the following to polly 3.4? >> >> Index: CMakeLists.txt >> =================================================================== >> --- CMakeLists.txt (revision 195142) >> +++ CMakeLists.txt (working

On 11/20/2013 04:50 PM, Jack Howarth wrote: > On Tue, Nov 19, 2013 at 12:07:18PM +0100, Tobias Grosser wrote: >> On 11/19/2013 08:50 PM, Jack Howarth wrote: >>> Tobias, >>> Can we add something like the following to polly 3.4? >>> >>> Index: CMakeLists.txt >>> ===================================================================

Tobias, Can we add something like the following to polly 3.4? Index: CMakeLists.txt =================================================================== --- CMakeLists.txt (revision 195142) +++ CMakeLists.txt (working copy) @@ -81,9 +81,14 @@ set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PAT FIND_PACKAGE(Isl REQUIRED) FIND_PACKAGE(Gmp REQUIRED) -FIND_PACKAGE(Cloog) FIND_PACKAGE(Pluto)

On 11/19/2013 08:50 PM, Jack Howarth wrote: > Tobias, > Can we add something like the following to polly 3.4? > > Index: CMakeLists.txt > =================================================================== > --- CMakeLists.txt (revision 195142) > +++ CMakeLists.txt (working copy) > @@ -81,9 +81,14 @@ set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PAT > > FIND_PACKAGE(Isl

On Wed, Nov 20, 2013 at 05:00:44PM +0100, Tobias Grosser wrote: > On 11/20/2013 04:50 PM, Jack Howarth wrote: >> On Tue, Nov 19, 2013 at 12:07:18PM +0100, Tobias Grosser wrote: >>> On 11/19/2013 08:50 PM, Jack Howarth wrote: >>>> Tobias, >>>> Can we add something like the following to polly 3.4? >>>> >>>> Index: CMakeLists.txt