[PATCH] D41420: [LV][VPlan] NFC patch to move LoopVectorizationPlanner class out of LoopVectorize.cpp

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 19 16:18:02 PST 2017

hsaito created this revision.
hsaito added reviewers: mkuper, hfinkel.
Herald added a subscriber: bollu.

Another small step forward to move VPlan stuff outside of LoopVectorize.cpp.

1. VPlanBuilder.h is renamed to LoopVectorizationPlanner.h
2. LoopVectorizationPlanner class is moved from LoopVectorize.cpp to LoopVectorizationPlanner.h
3. LoopVectorizationCostModel::VectorizationFactor class is moved to LoopVectorizationPlanner.h (used by the planner class) --- this needs further streamlining work in later patches and thus all I did was take it out of the CostModel class and moved to the header file.
4. The callback function had to stay inside LoopVectorize.cpp since it calls an InnerLoopVectorizer member function declared in it.

Next Steps: Make InnerLoopVectorizer, LoopVectorizationCostModel, and other classes more modular and more aligned with VPlan direction, in small increments.

Previous Step was: https://reviews.llvm.org/D41045.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41420.127616.patch
Type: text/x-patch
Size: 23406 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171220/e33e8738/attachment.bin>

More information about the llvm-commits mailing list