[PATCH] D76985: [LVI] Don't require DominatorTree in LVI (NFC)
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 18 01:34:03 PDT 2020
fhahn added inline comments.
================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:458
// Flush only the Dominator Tree.
DTU->getDomTree();
- LVI->enableDT();
----------------
I think we flushed the DT updates here for LVI only? It would be good to drop this here (less frequent applying of updates should be quicker). If it is required somewhere else, we should move it there.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76985/new/
https://reviews.llvm.org/D76985
More information about the llvm-commits
mailing list