[PATCH] D15559: [SCEVExpander] Make findExistingExpansion smarter

Junmo Park via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 23 03:16:16 PST 2015


flyingforyou retitled this revision from "[LoopUnrollRuntime] Do unroll when IV's setp is one or minus one" to "[SCEVExpander] Make findExistingExpansion smarter".
flyingforyou updated the summary for this revision.
flyingforyou updated this revision to Diff 43517.
flyingforyou marked an inline comment as done.
flyingforyou added a comment.
Herald added a subscriber: sanjoy.

Addressed Sanjoy's comment.

Following your suggestion, I realized that checing IV's step is not necessary any more.

I really appreciate your review.


http://reviews.llvm.org/D15559

Files:
  lib/Analysis/ScalarEvolutionExpander.cpp
  lib/Transforms/Utils/LoopUnrollRuntime.cpp
  test/Transforms/LoopUnroll/high-cost-trip-count-computation.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15559.43517.patch
Type: text/x-patch
Size: 5784 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151223/2f3303e5/attachment.bin>


More information about the llvm-commits mailing list