[PATCH] D33320: [SLP] Improve comments and naming of functions/variables/members, NFC.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 2 13:08:02 PDT 2017


ABataev added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:4782-4783
+  // horizontal reduction.
+  // INTERRUPT THE PROCESS IF THE Root INSTRUCITON ITSELF WAS VECTORIZED OR ALL
+  // SUB-TREES NOT HIGHER THAN RecursionMaxDepth WERE ANALYZED/VECTORIZED.
+  SmallVector<std::pair<WeakVH, unsigned>, 8> Stack(1, {Root, 0});
----------------
anemet wrote:
> Something got messed up with upper/lowercase here.
I tried to keep the original names of the variables. Should I capitalize them too?


https://reviews.llvm.org/D33320





More information about the llvm-commits mailing list