[PATCH] D87895: [SyntaxTree] Test for '\' inside token.

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 18 04:36:28 PDT 2020


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:173
+      R"cpp(
+in/
+t a;
----------------
That looks like the wrong slash to me (forward instead of backward).

To avoid possible confusion with line continuations in the source file itself, I'd suggest to write it as a regular string literal:

"in\\t a;"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87895



More information about the cfe-commits mailing list