[Polly] Replace the Dependences::isParallelDimension by the IslAst parallelism check
jdoerfert at codeaurora.org
Thu Jul 24 10:07:23 PDT 2014
I think I got this now. I will test it tonight myself if nobody tells me it
is fine as it is.
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
The Linux Foundation
From: Tobias Grosser [mailto:tobias at grosser.es]
Sent: Wednesday, July 23, 2014 10:48 PM
To: Johannes Doerfert; llvm-commits at cs.uiuc.edu
Subject: Re: [Polly] Replace the Dependences::isParallelDimension by the
IslAst parallelism check
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:
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:
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 10731 bytes
Desc: not available
More information about the llvm-commits