[PATCH] D32200: [LV] Refactor ILV.vectorize[Loop]() by introducing LVP.executePlan()

Ayal Zaks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 13:30:52 PDT 2017


Ayal added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:472
 
-  /// A helper function to vectorize a single instruction within the innermost
-  /// loop.
+public:
+  /// A function to vectorize a single instruction within the innermost loop.
----------------
mkuper wrote:
> Can you move this into the public part above, instead of adding one here?
Sure, will do.
This way hopefully conveyed the diff more easily.


https://reviews.llvm.org/D32200





More information about the llvm-commits mailing list