[PATCH] D23075: [IndVarSimplify] Extend trip count instead of truncating IV in LFTR, when original IV does not overflow

[Ehsan Amiri via llvm-commits]

amehsan updated this revision to Diff 67163.
amehsan added a comment.

added new testcases.

With the new implementation, (that we use zext if legal) two of the testcases no longer fail. So I have excluded those changes. Most minor comments on the code was also applied, except s/Extended/ExtendedIV/ as we are extending trip count, and I thought ExtendedTripCount is too long a variable name without carrying much information given the comments in the code and overall code size for this logic.


https://reviews.llvm.org/D23075

Files:
  lib/Transforms/Scalar/IndVarSimplify.cpp
  test/Transforms/IndVarSimplify/elim-extend.ll
  test/Transforms/IndVarSimplify/lftr-wide-trip-count.ll
  test/Transforms/IndVarSimplify/ult-sub-to-eq.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23075.67163.patch
Type: text/x-patch
Size: 9356 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160808/3c9a8c0c/attachment.bin>