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

Ben Langmuir via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 14 13:40:56 PDT 2017


benlangmuir added a comment.

> What do you think about a hybrid approach: Don't suppress diagnostics in the placeholder range when we handle the placeholders in parser/sema, but do suppress the range when the placeholder isn't explicitly handled?

I'd really rather not plumb this into the diagnostics engine at all if we can avoid it.  Are there cases we somehow cannot handle using the explicit checks, or is it just a question of finding all the cases?


Repository:
  rL LLVM

https://reviews.llvm.org/D32081





More information about the cfe-commits mailing list