[PATCH] D70856: [Syntax] Build nodes for simple cases of top level declarations

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 9 06:52:33 PST 2019


ilya-biryukov updated this revision to Diff 232840.
ilya-biryukov marked 5 inline comments as done.
ilya-biryukov added a comment.

- Remove the duplicate test
- Add 'message' to the node for static_assert
- Handle nested namespace definitions, add them to the tests


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70856/new/

https://reviews.llvm.org/D70856

Files:
  clang/include/clang/Tooling/Syntax/Nodes.h
  clang/lib/Tooling/Syntax/BuildTree.cpp
  clang/lib/Tooling/Syntax/Nodes.cpp
  clang/unittests/Tooling/Syntax/TreeTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70856.232840.patch
Type: text/x-patch
Size: 12623 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191209/dbe57e43/attachment-0001.bin>


More information about the cfe-commits mailing list