[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