[PATCH] D55649: [clangd] Enable cross-namespace completions by default in clangd

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 13 06:53:44 PST 2018


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


================
Comment at: clangd/tool/ClangdMain.cpp:144
         "can insert scope qualifiers."),
-    cl::init(false), cl::Hidden);
+    cl::init(true));
 
----------------
kadircet wrote:
> why not keep it hidden ?
We hid the flag because the feature was experimental. Now that it's the default, I think it makes sense to expose the flag.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D55649





More information about the cfe-commits mailing list