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

[Tobias Grosser via llvm-commits]

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.

Best,
Tobias


Repository:
  rL LLVM

http://reviews.llvm.org/D17381