[PATCH] D12062: Allow values to cause memory accesses

[Johannes Doerfert via llvm-commits]

jdoerfert added a comment.

We could have both, an access instruction as well as an access value, that way we keep the best of both worlds. I will update the patch as I think it might be worth it.


================
Comment at: lib/Analysis/ScopInfo.cpp:974
@@ -978,1 +973,3 @@
+    if (StoreMA->isScalar())
+      continue;
     if (StoreMA->isRead())
----------------
grosser wrote:
> This change is also confusing. Does this mean we will not support scalar reductions any more?
We never did in upstream Polly.


http://reviews.llvm.org/D12062