search for: cloog

...4 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". >>> >>> Dear Star Tan, >>> >>> thanks a lot for the patch. It looks very reasonable, but I am wondering >>> why it was not needed before or what problem it fixes exa...

..."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 reasonable, but I am wondering >> >>> why it was n...

...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 reasonable, but I am wondering > >>> why it was not needed before or...

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 reasonable, but I am wondering >why it was not needed before or what problem it fixes exactly. Could you >add some information...

...================================= >> --- 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) >> >> +option(POLLY_USE_CLOOG "Build Polly with Cloog support" OFF) >> +if (POLLY_USE_CLOOG) >> + # Build Cloog support in Polly (default is ISL-only). >> + FIND_PACKAGE(Cloog) >> +endif(POLLY_USE_CLOOG) > > Hi Ja...

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". >> >> Dear Star Tan, >> >> thanks a lot for the patch. It looks very reasonable, but I am wondering >> why it was not needed before or what problem it fixes exactly. Could you >...

...keLists.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) >>>> >>>> +option(POLLY_USE_CLOOG "Build Polly with Cloog support" OFF) >>>> +if (POLLY_USE_CLOOG) >>>> + # Build Cloog support in Polly (default is ISL-only). >>>> + FIND_PACKAGE(Cloog)...

...=== >>> --- 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) >>> >>> +option(POLLY_USE_CLOOG "Build Polly with Cloog support" OFF) >>> +if (POLLY_USE_CLOOG) >>> + # Build Cloog support in Polly (default is ISL-only). >>> + FIND_PACKAGE(Cloog) >>> +endif(POL...

...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) +option(POLLY_USE_CLOOG "Build Polly with Cloog support" OFF) +if (POLLY_USE_CLOOG) + # Build Cloog support in Polly (default is ISL-only). + FIND_PACKAGE(Cloog) +endif(POLLY_USE_CLOOG) + option(POLLY_ENABLE_GPGPU_CODEGEN "Enable GPGPU code generation featu...

...============================================================= > --- 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) > > +option(POLLY_USE_CLOOG "Build Polly with Cloog support" OFF) > +if (POLLY_USE_CLOOG) > + # Build Cloog support in Polly (default is ISL-only). > + FIND_PACKAGE(Cloog) > +endif(POLLY_USE_CLOOG) Hi Jack, thanks for writing a patch for y...

Hi Tobi, when polly is configured with isl and without cloog, make polly-test does not pass cleanly as there are several testcases that use a pass flag that does not exist in this configuration: -polly-cloog. I was thinking to address this problem by renaming -polly-cloog and -polly-codegen-isl to -polly-codegen and making all these tests using -polly-codeg...

Tobi, What is the situation with -polly-codegen-isl in polly svn? Should we be testing polly without cloog as the default configuration or will cloog use not be deprecated in 3.3? Jack

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...

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 directory, so if you want I can > >disable all the CodeGen directory (and subdirectories) when Cloog is not > >available. Note that in my first patch...

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 that will adapt mos...

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 in my first patch I have moved only those tests not working w...

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 CodeGen directory (and subdirectories) when Cloog is not > available. Note that in my first patch I have moved only th...

...2 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. >>> >>> 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 tha...

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 > > dependent on Cloog to be conditionally executed to CLOOG_FOUND. I am preparing > > another...

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 th...