[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