[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:21:49 PDT 2019


ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.

LGTM (see the comment about a typo, though)



================
Comment at: clangd/index/SymbolCollector.cpp:669
+    return false;
+  return Line.contains_lower("includ");
+}
----------------
A typo? Should it be `include`.


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