[PATCH] D65486: [clangd] Ignore semicolons, whitespace, and comments in SelectionTree.

Shaurya Gupta via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 31 07:15:14 PDT 2019


SureYeaah added inline comments.


================
Comment at: clang-tools-extra/clangd/Selection.cpp:50
+      S.StartOffset = SM.getFileOffset(Tok.location());
+      S.EndOffset = S.StartOffset + Tok.length();
+      if (S.StartOffset >= SelBegin && S.EndOffset <= SelEnd)
----------------
Would this work correctly for nested templates? Or do we need to use the specialized token length function that we use for toHalfOpenFileRange?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65486





More information about the cfe-commits mailing list