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

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 08:20:35 PST 2016

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.

Also, if you are interested to eliminate - based on this patch - the reduction dependence detection. This would be of great value.




More information about the llvm-commits mailing list