[PATCH] D11878: [Polly] Refactor buildScop

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 9 04:50:26 PDT 2015


jdoerfert accepted this revision.
jdoerfert added a reviewer: jdoerfert.
jdoerfert added a comment.
This revision is now accepted and ready to land.

1 Comment but otherwise LGTM.


================
Comment at: include/polly/ScopInfo.h:914
@@ +913,3 @@
+  __isl_give isl_schedule *
+  ScheduleCombineSequence(__isl_take isl_schedule *Prev,
+                          __isl_take isl_schedule *Succ);
----------------
Usually method names start with a small letter, what about combineInSequence(..., ...) ?


http://reviews.llvm.org/D11878





More information about the llvm-commits mailing list