[PATCH] D16623: Factor out UnrollAnalyzer to Analysis, and add unit tests for it.

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 20:50:30 PST 2016


mzolotukhin created this revision.
mzolotukhin added reviewers: chandlerc, hfinkel.
mzolotukhin added a subscriber: llvm-commits.
Herald added a subscriber: sanjoy.

Unrolling Analyzer is already pretty complicated, and it becomes harder and harder to exercise it with usual IR tests, as with them we can only check the final decision: whether the loop is unrolled or not. This change factors this framework out from LoopUnrollPass to analyses, which allows to use unit tests.
The change itself is supposed to be NFC, except adding a couple of tests.

I plan to add more tests as I add new functionality and find/fix bugs.

http://reviews.llvm.org/D16623

Files:
  include/llvm/Analysis/LoopUnrollAnalyzer.h
  lib/Analysis/CMakeLists.txt
  lib/Analysis/LoopUnrollAnalyzer.cpp
  lib/Transforms/Scalar/LoopUnrollPass.cpp
  unittests/Analysis/CMakeLists.txt
  unittests/Analysis/UnrollAnalyzer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16623.46098.patch
Type: text/x-patch
Size: 24575 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160127/a04c7476/attachment.bin>


More information about the llvm-commits mailing list