similar to: [LLVMdev] [Polly] Update lit config for Cloog

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

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

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

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

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-codegen. Another option is to disable these

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

Tobias, Would it be possible to add a cmake option to disable the use of cloog in the polly build for 3.4? Currently the toplevel CMakeLists.txt in polly has... FIND_PACKAGE(Isl REQUIRED) FIND_PACKAGE(Gmp REQUIRED) FIND_PACKAGE(Cloog) FIND_PACKAGE(Pluto) with no cmake option to avoid executing... FIND_PACKAGE(Cloog) I have been deinstalling the fink cloog packages, so 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

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

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

Hello all, The performance comparison between Polly's Cloog and ISL code generator is posted on http://188.40.87.11:8000/db_default/v4/nts/59?compare_to=58&baseline=58 It seems their execution-time performance are comparable: Performance Regressions - Execution Time  (ISL over Cloog) MultiSource/Benchmarks/TSVC/ControlFlow-flt/ControlFlow-flt 8.49%

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

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