[PATCH] D40146: [JumpThreading] Deferred preservation of DT and LVI across the pass.

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 8 13:17:39 PST 2017


kuhar added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/JumpThreading.cpp:289
   auto AA = &getAnalysis<AAResultsWrapperPass>().getAAResults();
+  DeferredDominance *DDT = new DeferredDominance(*DT);
   std::unique_ptr<BlockFrequencyInfo> BFI;
----------------
One remark: why new here instead of just putting it on the stack? If you need to pass the ownership around, who deallocates it? 


https://reviews.llvm.org/D40146





More information about the llvm-commits mailing list