xazax.hun accepted this revision. xazax.hun added a comment. This revision is now accepted and ready to land.
Looks good to me, some nits inline. ================ Comment at: clang/unittests/Analysis/CFGBuildResult.h:1 +//===- unittests/Analysis/CFGTest.cpp - CFG tests -------------------------===// +// ---------------- Filename not updated? ================ Comment at: clang/unittests/Analysis/CFGDominatorTree.cpp:1 +//===- unittests/Analysis/CFGTest.cpp - CFG tests -------------------------===// +// ---------------- Filename not updated? ================ Comment at: clang/unittests/Analysis/CFGDominatorTree.cpp:13 +#include "clang/Analysis/CFG.h" +#include "clang/Tooling/Tooling.h" +#include "gtest/gtest.h" ---------------- Do you need the Tooling header here? 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
