[PATCH] D43173: [CallSiteSplitting] Preserve DominatorTreeAnalysis.

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 13 10:21:56 PST 2018


dberlin added a comment.

So this looks right to me.
I'm a little worried about how we are having to modify tons and tons of functions to push these through.
When we get to postdominator updates, we'll have to do the same thing.

At some point, if what we are trying to do is preserve analysis, i wonder if we shouldn't be passing analysismanager's around or something.


https://reviews.llvm.org/D43173





More information about the llvm-commits mailing list