[PATCH] D61120: [clangd] Optimize "don't include me" check.

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 25 03:31:04 PDT 2019


ilya-biryukov added inline comments.


================
Comment at: clangd/index/SymbolCollector.cpp:669
+    return false;
+  return Line.contains_lower("includ");
+}
----------------
sammccall wrote:
> ilya-biryukov wrote:
> > A typo? Should it be `include`.
> it's meant to match "include" or "including". I'll add a comment.
Makes sense. Comment would work, thanks!


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D61120





More information about the cfe-commits mailing list