[PATCH] D45420: Moved RecurrenceDescriptor/LoopDescriptor from Transform/Utils/LoopUtils.* to Analysis tree
Hideki Saito via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 8 14:18:57 PDT 2018
hsaito created this revision.
hsaito added reviewers: karthikthecool, TylerNowicki, jmolloy.
Herald added subscribers: llvm-commits, mgorny.
I'd like to move LoopVectorizationLegality in Transform/Vectorize/LoopVectorize.cpp to Analysis tree so that it is reusable from outside of loop vectorizer (e.g., other analysis/transformation to check whether loop is vectorizable and behave differently).
RecurrenceDescriptor and InductionDescriptor classes are used by LoopVectorizationLegality class. Nature of all those classes are indeed
analysis and thus Analysis tree is a better place to house them.
Includes and forward declared classes are also cleaned up. I really didn't want to include IRBuilder.h from a header --- since it includes a lot of other headers, but it looked like forward declaring IRBuilder wasn't too appealing to me.
Repository:
rL LLVM
https://reviews.llvm.org/D45420
Files:
include/llvm/Analysis/Utils/LoopUtils.h
include/llvm/Transforms/Utils/LoopUtils.h
lib/Analysis/CMakeLists.txt
lib/Analysis/LoopUtils.cpp
lib/Transforms/Scalar/LICM.cpp
lib/Transforms/Scalar/LoopIdiomRecognize.cpp
lib/Transforms/Scalar/LoopUnswitch.cpp
lib/Transforms/Utils/LoopUtils.cpp
lib/Transforms/Vectorize/LoopVectorize.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D45420.141572.patch
Type: text/x-patch
Size: 122869 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180408/584aeecb/attachment-0001.bin>
More information about the llvm-commits
mailing list