[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