[PATCH] D13676: [Polly] Store leading and trailing memory accesses separately

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 5 08:25:58 PST 2015


grosser added a comment.

Hi Michael,

from my perspective the patch looks good to me, except the name concerns Johannes raised. Having looked into some of the other patches, I see this small (but apparently  recurring) issue again. I started a mailing list discussion to hopefully resolve this once and for all.

Best,
Tobias


================
Comment at: include/polly/ScopInfo.h:685
@@ -683,1 +684,3 @@
+  bool isScalar() const { return Origin == SCALAR; }
+
   /// @brief Is this MemoryAccess modeling special PHI node accesses?
----------------
This change is unrelated. Please commit it separately.


http://reviews.llvm.org/D13676





More information about the llvm-commits mailing list