ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land.
LGTM! ================ Comment at: unittests/clangd/ClangdUnitTests.cpp:268 + auto AST = TU.build(); + ASSERT_EQ(AST.getLocalTopLevelDecls().size(), 1u); + NamedDecl *ND = dyn_cast<NamedDecl>(AST.getLocalTopLevelDecls().front()); ---------------- NIT: maybe use `ASSEET_THAT(AST.getLocalTopLevelDecls(), ElementsAre(...))` Unless that's too hard for some reason. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D54303 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
