[PATCH] D32081: Add support for editor placeholders to Clang's lexer

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 14 07:41:39 PDT 2017


arphaman updated this revision to Diff 95297.
arphaman added a comment.

I've decided to treat each placeholder as an identifier token (like Swift) instead of just lexing the contents. This will prevent spurious errors for declaration name placeholders that have spaces or hyphens inside.


Repository:
  rL LLVM

https://reviews.llvm.org/D32081

Files:
  include/clang/Basic/Diagnostic.h
  include/clang/Basic/DiagnosticLexKinds.td
  include/clang/Basic/DiagnosticOptions.def
  include/clang/Driver/Options.td
  include/clang/Lex/Lexer.h
  lib/Basic/Diagnostic.cpp
  lib/Basic/DiagnosticIDs.cpp
  lib/Driver/ToolChains/Clang.cpp
  lib/Frontend/CompilerInvocation.cpp
  lib/Lex/Lexer.cpp
  test/Driver/clang_f_opts.c
  test/Lexer/editor-placeholder.cpp
  test/Parser/placeholder-recovery.m

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32081.95297.patch
Type: text/x-patch
Size: 11481 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170414/6f6ed1d9/attachment.bin>


More information about the cfe-commits mailing list