[PATCH] D88669: [ValueTracking] Add tracking of the alignment assume bundle
Shimin Cui via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 14 09:54:23 PDT 2020
scui updated this revision to Diff 298169.
scui added a comment.
Rewrite the code using getKnowledgeValidInContext. The code is much cleaner than the previous version, but the result computed is less accurate if the size of the alignment bundle is three. I'll leave the improvement for the future, together with the improvement of value tracking of GEP (an example is that, with the future improvement, we can fold away the branch "br i1 %3, ...") in f2 of the added case assume-align.ll).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88669/new/
https://reviews.llvm.org/D88669
Files:
llvm/lib/Analysis/ValueTracking.cpp
llvm/test/Analysis/BasicAA/featuretest.ll
llvm/test/Analysis/ValueTracking/assume-queries-counter.ll
llvm/test/Transforms/InstCombine/assume-align.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88669.298169.patch
Type: text/x-patch
Size: 5728 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201014/b3fc254a/attachment.bin>
More information about the llvm-commits
mailing list