[PATCH] D139926: [clangd] Add semantic token for angle brackets
Nathan Ridge via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 24 00:43:18 PST 2023
nridge added a comment.
One other thought that has occurred to me is that as we add more semantic tolens for punctuation, the test cases in `GetsCorrectTokens` become harder to read.
What would you think about omitting punctuation tokens in the `GetsCorrectTokens` test cases (both `Operator` and `Bracket`, and any future ones), similarly to the way we already omit scope modifiers like `_classScope`, and have another test case like `GetsCorrectPunctuationTokens` where we put specifically test cases that test for punctuation tokens?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139926/new/
https://reviews.llvm.org/D139926
More information about the cfe-commits
mailing list