[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