Szelethus marked 5 inline comments as done. Szelethus added inline comments.
================ Comment at: clang/unittests/Analysis/CFGDominatorTree.cpp:95 + CFGPostDomTree PostDom; + PostDom.buildDominatorTree(cfg); + ---------------- kuhar wrote: > Why not have a constructor that takes the cfg and constructs a domtree > straight away? But this should probably go into a separate patch. Yea, it's been bothering me for a while, but apparently not enough to fix it (just yet!). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62611/new/ https://reviews.llvm.org/D62611 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
