sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land.
================ Comment at: unittests/clangd/CodeCompleteTests.cpp:2180 + clangd::CodeCompleteOptions Opts; + Opts.AllScopes = true; + auto Results = completions(R"cpp( ---------------- Any need to actually enable this option, or just copied from above test? (If not, no need to pass opts at all, it'll be defaulted). Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53192 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
