[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...
Size: 23406 bytes
Desc: not available
More information about the llvm-commits