[PATCH] D17381: [Polly] [RFC] Fine-grain dependency analysis

Hongbin Zheng via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 08:31:43 PST 2016


On Fri, Feb 26, 2016 at 12:20 AM, Tobias Grosser <tobias at grosser.es> wrote:

> grosser added a comment.
>
> Hi ether,
>
> I do not think that there is a need to factor this into another pass. In
> fact, I think for the state we are currently at, the 'flag' solution works
> well as it shows what can be done and allows us to test the functionality.
> If we later need this functionality (simultaneously) in different passes,
> we can always refactor to make it easier accessible. From my point of view,
> this patch is good to go. However, I would suggest to add a reference-type
> test case.
>
Sure


> Also, if you are interested to eliminate - based on this patch - the
> reduction dependence detection. This would be of great value.
>
I can think about this. We can discuss this latter

>
> Best,
> Tobias
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D17381
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160226/c75792ed/attachment.html>


More information about the llvm-commits mailing list