[PATCH] [Polly][Refactor][NFC] Simplify and clean the handling of (new) access relations
David Peixotto
dpeixott at codeaurora.org
Thu Oct 9 12:10:06 PDT 2014
================
Comment at: include/polly/ScopInfo.h:245
@@ +244,3 @@
+ /// @brief Get the new access function imported or set by a pass
+ isl_map *getNewAccessRelation() const;
+
----------------
Does these two methods need to be public? If not it would be good to make them private to avoid confusion over which method to call (e.g. `getAccessRelation`, `getNewAccessRelation`, `getNewestAccessRelation`). I'm not too familiar with the API, so do whatever makes sense.
Otherwise, LGTM.
http://reviews.llvm.org/D5707
More information about the llvm-commits
mailing list