[Polly] Replace the Dependences::isParallelDimension by the IslAst parallelism check

Tobias Grosser tobias at grosser.es
Wed Jul 23 22:47:58 PDT 2014


On 24/07/2014 01:40, Johannes Doerfert wrote:
> This time I attached the patch.

I just applied this patch and I get the following compile error:

/home/grosser/Projects/polly/git/tools/polly/lib/CodeGen/CodeGeneration.cpp:884:8: 
error: member reference type 'polly::Dependences' is not a pointer
       D->getDependences(Dependences::TYPE_RAW | Dependences::TYPE_WAW |
       ~^
1 error generated

If I fix this one, many CLooG test cases fail with:

/home/grosser/Projects/polly/tools/src/cloog/isl/isl_ctx.c:245: isl_ctx
freed, but some objects still reference it 0  libLLVMSupport.so
0x00007f0ac46dc1ce llvm::sys::PrintStackTrace(_IO_FILE*) + 46

I think you really need to compile and test your code before submitting
(I know this adds overhead. If we get the OpenMP patches, we can remove
the dependency to CLooG entierly)

I also looked over the patch. It looks like a very nice cleanup.

Cheers,
Tobias





More information about the llvm-commits mailing list