[PATCH] D52016: [clangd] Don't create child AND and OR iterators with one posting list

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov added inline comments.


================
Comment at: clang-tools-extra/clangd/index/dex/Dex.cpp:148
+  if (TrigramIterators.size() > 1)
     TopLevelChildren.push_back(createAnd(move(TrigramIterators)));
+  else if (TrigramIterators.size() == 1)
----------------
Maybe special-case a single-iterator case in `createAnd` instead? Same with 
`createOr`.
Any cons to doing so?


https://reviews.llvm.org/D52016



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits