[PATCH] D51153: Break LoopUtils into an Analysis file.

Vikram TV via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 22 23:15:17 PDT 2018


tvvikram created this revision.
tvvikram added reviewers: dmgreen, llvm-commits.
Herald added a subscriber: mgorny.

The InductionDescriptor and RecurrenceDescriptor classes basically analyzes the IR to identify the respective IVs. So, it is better to have them in the "Analysis" directory instead of the "Transforms" directory.

The rationale for this is to make the Induction and Recurrence descriptor classes available for analysis passes. Currently including them in an analysis pass produces link error (http://lists.llvm.org/pipermail/llvm-dev/2018-July/124456.html).

Base classes for the above classes are in the new file - Analysis/LoopAnalysisUtils.h. This file is included in Transforms/LoopUtils.h. Respective definitions reside in Analysis/LoopAnalysisUtils.cpp and Transforms/LoopUtils.cpp.


Repository:
  rL LLVM

https://reviews.llvm.org/D51153

Files:
  Analysis/CMakeLists.txt
  Analysis/LoopAnalysisUtils.cpp
  Transforms/Utils/LoopUtils.cpp
  llvm/Analysis/LoopAnalysisUtils.h
  llvm/Transforms/Utils/LoopUtils.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51153.162137.patch
Type: text/x-patch
Size: 111625 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180823/de21b562/attachment-0001.bin>


More information about the llvm-commits mailing list