[PATCH] D54312: [CSP, Cloning] Update DuplicateInstructionsInSplitBetween to use DomTreeUpdater.
Jakub Kuderski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 9 07:15:32 PST 2018
kuhar added inline comments.
================
Comment at: lib/Transforms/Scalar/CallSiteSplitting.cpp:495
static bool tryToSplitCallSite(CallSite CS, TargetTransformInfo &TTI,
- DominatorTree &DT) {
+ DominatorTree &DT, DomTreeUpdater &DTU) {
if (!CS.arg_size() || !canSplitCallSite(CS, TTI))
----------------
You can have just DTU and ask it for a DT: `DTU.getDomTree()`. This was it's easier to make sure you are not making any queries to an outdated DomTree.
https://reviews.llvm.org/D54312
More information about the llvm-commits
mailing list