[PATCH] D47259: [IPSCCP,PM] Preserve DT in the new pass manager.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: lib/Transforms/IPO/SCCP.cpp:24
+  PreservedAnalyses PA;
+  PA.preserve<DominatorTreeAnalysis>();
+  PA.preserve<FunctionAnalysisManagerModuleProxy>();
----------------
NutshellySima wrote:
> It is also possible to preserve PostDominatorTree here trivially using `DTU(DT, PDT, ...)` and `auto* PDT = FAM.getCachedResult`.
IPSCCP currently runs relatively early, but if beneficial it can be added in a follow up patch quite easily by extending the AnalysisResultsForFn struct.


https://reviews.llvm.org/D47259