[PATCH] D17632: [LoopUnrollAnalyzer] Check that we're using SCEV for the same loop we're simulating.

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 17:59:03 PST 2016


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

Check that we're using SCEV for the same loop we're simulating. Otherwise, we might try to use the iteration number of the current loop in SCEV expressions for inner/outer loops IVs, which is clearly incorrect.

http://reviews.llvm.org/D17632

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17632.49135.patch
Type: text/x-patch
Size: 4593 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160226/0fd9da07/attachment.bin>


More information about the llvm-commits mailing list