[PATCH] D101851: Make clangd CompletionModel not depend on directory layout.

Harald van Dijk via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 4 15:18:46 PDT 2021


hvdijk marked an inline comment as done.
hvdijk added inline comments.


================
Comment at: clang-tools-extra/clangd/quality/CompletionModel.cmake:7
 # namespace-qualified class name.
+set(CLANGD_QUALITY_DIR ${CMAKE_CURRENT_LIST_DIR})
 function(gen_decision_forest model filename cpp_class)
----------------
usaxena95 wrote:
> nit: Directly refer to code generator instead of storing quality dir.
> `set(CLANGD_COMPLETION_MODEL_COMPILER ${CMAKE_CURRENT_LIST_DIR}/CompletionModelCodegen.py)`
Alright, done. Testing in progress just to be on the safe side. We could also get rid of `model_compiler` and use `${CLANGD_COMPLETION_MODEL_COMPILER}` directly, but I felt that looked bad due to the conflicting variable naming styles and the long lines, so I left that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101851



More information about the cfe-commits mailing list