[LLVMdev] [polly] removing cloog dependence in the testsuite

Tobias Grosser tobias at grosser.es
Mon Dec 3 09:15:50 PST 2012


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 those tests not
> working without cloog, and there are several testcases left in CodeGen that can
> still be executed with ISL's codegen.

I did not realize there are tests that work for both.
What are those tests doing?

>> You can add the new isl test cases to  test/IslCodeGen.
>>
>
> That's what patch 3 does, except for a minor rename to avoid a redundant CodeGen
> in the dir names.

I was basically aiming for having one test directory for one LLVM-IR 
pass. Instead of having shared test cases, I would rather copy those 
tests to the isl directory.

Cheers
Tobi



More information about the llvm-dev mailing list