[PATCH] D59887: [Syntax] Introduce TokenBuffer, start clangToolingSyntax library

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov marked 2 inline comments as done.
ilya-biryukov added inline comments.


================
Comment at: include/clang/Tooling/Syntax/Tokens.h:206
+  ///     DECL(a);
+  /// spelledTokens() returns {"#", "define", "DECL", "(", "name", ")", "eof"}.
+  /// FIXME: we do not yet store tokens of directives, like #include, #define,
----------------
nridge wrote:
> Is this comment correct?
> 
> If so:
>   * Why are the tokens "int", "name", "=", "10" not included?
>   * Why are the tokens `"DECL", "(", "a", ")" not included?
> 
It isn't. Thanks for catching that.
This patch went through so many revisions, it was close to impossible to track this down.


Repository:
  rC Clang

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

https://reviews.llvm.org/D59887