gribozavr2 accepted this revision.
gribozavr2 added inline comments.
This revision is now accepted and ready to land.
================
Comment at: clang/unittests/Tooling/Syntax/BuildTreeTest.cpp:16
+namespace clang {
+namespace syntax_test {
namespace {
----------------
Ditto, please use namespace clang::syntax.
================
Comment at: clang/unittests/Tooling/Syntax/TreeTestBase.h:44
+namespace clang {
+namespace syntax_test {
+static ArrayRef<syntax::Token> tokens(syntax::Node *N) {
----------------
Please use the same namespace that the non-test code is using.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85897/new/
https://reviews.llvm.org/D85897
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
