[PATCH] D75384: OpenMP for loop fusion

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 16 16:57:24 PDT 2020


jdoerfert added a comment.

Any update on this?



================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:153
+
+  /// The following function is the main pipeline for  the whole process
+  bool deleteStaticScheduleCalls() {
----------------
abidmalikwaterloo wrote:
> jdoerfert wrote:
> > Nit: two spaces, if you write sentences add the '.'.
> not sure what does this means.   I should add ".. at the end of each comment line????
You should a dot, ".", at the end of a full sentence.



CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75384/new/

https://reviews.llvm.org/D75384





More information about the llvm-commits mailing list