search for: islcodegen

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

...s (5): > Polly :: CodeGen/do_pluto_matmult.ll > Polly :: CodeGen/single_do_loop_int_max_iterations.ll > Polly :: CodeGen/single_do_loop_scev_replace.ll I have some guesses for these, but it would be good to get run llvm-lit -v on these three test cases. > Polly :: IslCodeGen/single_loop_param_less_equal.ll > Polly :: IslCodeGen/single_loop_param_less_than.ll This is due to the use of an old isl version. I will change configure to fail if the isl version used is too old. > Are these darwin-specific or due to the fact that I am using isl 0.10 and > cloog...

...polly-test is passing with both configurations > enabling > either the cloog codegen and the isl codegen. Very nice. Thanks for the work! > Is it ok to commit all these three patches? Patch two is fine. 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. You can add the new isl test cases to test/IslCodeGen. If you agree, such changes are preapproved here. Tobi

...nks for fixing this so quickly. Do you have any comments on failing tests we are seeing on x86_64-apple-darwin11? Failing Tests (5): Polly :: CodeGen/do_pluto_matmult.ll Polly :: CodeGen/single_do_loop_int_max_iterations.ll Polly :: CodeGen/single_do_loop_scev_replace.ll Polly :: IslCodeGen/single_loop_param_less_equal.ll Polly :: IslCodeGen/single_loop_param_less_than.ll Are these darwin-specific or due to the fact that I am using isl 0.10 and cloog 0.17 rather than their the upstream development svn? Let me know if any of those merit a bugzilla of their own. Jack &gt...

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

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

...do_pluto_matmult.ll >> Polly :: CodeGen/single_do_loop_int_max_iterations.ll >> Polly :: CodeGen/single_do_loop_scev_replace.ll > > I have some guesses for these, but it would be good to get run > > llvm-lit -v on these three test cases. > >> Polly :: IslCodeGen/single_loop_param_less_equal.ll >> Polly :: IslCodeGen/single_loop_param_less_than.ll Tobias, I am unclear on how I should modify the 'make polly-test' run in order for 'llvm-lit -v' to take effect and where to look for the additional information. Thanks in advance fo...

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

On 10/21/2012 11:46 AM, Jack Howarth wrote: > On Sun, Oct 21, 2012 at 02:35:49PM -0400, Jack Howarth wrote: >> On Sun, Oct 21, 2012 at 11:01:37AM -0700, Tobias Grosser wrote: >>> On 10/21/2012 10:57 AM, Jack Howarth wrote: >>>> On Sun, Oct 21, 2012 at 10:38:48AM -0700, Tobias Grosser wrote: >>>>> On 10/21/2012 09:13 AM, Jack Howarth wrote:

...Polly :: CodeGen/single_do_loop_int_max_iterations.ll >>> Polly :: CodeGen/single_do_loop_scev_replace.ll >> >> I have some guesses for these, but it would be good to get run >> >> llvm-lit -v on these three test cases. >> >>> Polly :: IslCodeGen/single_loop_param_less_equal.ll >>> Polly :: IslCodeGen/single_loop_param_less_than.ll > > Tobias, > I am unclear on how I should modify the 'make polly-test' run in order > for 'llvm-lit -v' to take effect and where to look for the additional informat...

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

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