[PATCH] D62939: Fix a bug w/inbounds invalidation in LFTR (was: Strengthen LFTR's ability to replace IVs)

[Philip Reames via Phabricator via llvm-commits]

reames updated this revision to Diff 203918.
reames retitled this revision from "Strengthen LFTR's ability to replace IVs" to "Fix a bug w/inbounds invalidation in LFTR (was: Strengthen LFTR's ability to replace IVs)".
reames edited the summary of this revision.
reames added a comment.

Minor rebase.  As pointed out by previous comments both from myself and Nikita, the entire approach here is broken.  This doesn't actually appear to be separable from D62936 <https://reviews.llvm.org/D62936>.  Given all the meaningful review has happened here, I'm going to repurpose this review for the bug fix and close that one.

For the moment though, this patch is known stale and needs reworked.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62939/new/

https://reviews.llvm.org/D62939

Files:
  lib/Transforms/Scalar/IndVarSimplify.cpp
  test/Transforms/IndVarSimplify/2011-11-01-lftrptr.ll
  test/Transforms/IndVarSimplify/lftr-dead-ivs.ll
  test/Transforms/IndVarSimplify/lftr-reuse.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62939.203918.patch
Type: text/x-patch
Size: 14091 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190610/2794a9dd/attachment.bin>