[PATCH] D76985: [LVI] Don't require DominatorTree in LVI (NFC)
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Mar 28 05:55:37 PDT 2020
nikic created this revision.
nikic added reviewers: reames, lebedev.ri.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
nikic added a parent revision: D76797: [LVI] Don't use dominator tree in isValidAssumeForContext().
After D76797 <https://reviews.llvm.org/D76797> there will no longer be any uses of the dominator tree in LVI, so we can remove it as a pass dependency, and also get rid of the dominator tree enabling/disabling logic in JumpThreading.
Apart from cleaning up the code, this also clarifies LVI cache consistency, in that the LVI cache can no longer depend on whether the DT was or wasn't enabled due to pending DT updates at any given time.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D76985
Files:
include/llvm/Analysis/LazyValueInfo.h
lib/Analysis/LazyValueInfo.cpp
lib/Transforms/Scalar/JumpThreading.cpp
lib/Transforms/Utils/LowerSwitch.cpp
test/Analysis/LazyValueAnalysis/invalidation.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76985.253333.patch
Type: text/x-patch
Size: 17532 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200328/ff2b3883/attachment-0001.bin>
More information about the llvm-commits
mailing list