[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