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

[Hideki Saito via Phabricator via llvm-commits]

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.


https://reviews.llvm.org/D41420

Files:
  lib/Transforms/Vectorize/LoopVectorizationPlanner.h
  lib/Transforms/Vectorize/LoopVectorize.cpp
  lib/Transforms/Vectorize/VPlanBuilder.h

-------------- 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>