[PATCH] D107145: clangd: Add new semantic token modifier "virtual"

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 30 03:14:56 PDT 2021


sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

The modifier seems reasonable to me.
The most important cases are callsites (where `virtual`) isn't written. Can you extend a test to cover some such case?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107145



More information about the cfe-commits mailing list