[PATCH] D103991: [SCEV] Move mustprogress based no-self-wrap logic so it applies to all exit conditions
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 18 08:39:50 PST 2021
reames updated this revision to Diff 388214.
reames retitled this revision from "[SCEV] Extend mustprogress reasoning to ne exit tests" to "[SCEV] Move mustprogress based no-self-wrap logic so it applies to all exit conditions".
reames edited the summary of this revision.
reames added reviewers: fhahn, lebedev.ri.
reames added a comment.
This update reworks the patch, and hopefully makes it a lot more obviously correct. The patch is restructured to purely pull code up from howManyLessThans into the caller so that it handles all condition codes. This does reduce the scope slightly, but I've included planned extensions in the review description which should cover all cases.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103991/new/
https://reviews.llvm.org/D103991
Files:
llvm/lib/Analysis/ScalarEvolution.cpp
llvm/test/Analysis/ScalarEvolution/ne-overflow.ll
llvm/test/Analysis/ScalarEvolution/trip-count-negative-stride.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D103991.388214.patch
Type: text/x-patch
Size: 7070 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211118/b493e8ae/attachment.bin>
More information about the llvm-commits
mailing list