[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