[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