[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