[PATCH] D43173: [CallSiteSplitting] Preserve DominatorTreeAnalysis.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 12 09:02:44 PST 2018


fhahn added inline comments.


================
Comment at: lib/Transforms/Scalar/CallSiteSplitting.cpp:410
   auto &TLI = AM.getResult<TargetLibraryAnalysis>(F);
+  auto &DT = AM.getResult<DominatorTreeAnalysis>(F);
 
----------------
junbuml wrote:
> Considering that this pass doesn't use DT, do we need to get DT to preserve DT ?
We need the current dominator tree, so we can update it. Not sure if there is a better way to get it though.


Repository:
  rL LLVM

https://reviews.llvm.org/D43173





More information about the llvm-commits mailing list