usaxena95 accepted this revision. usaxena95 added a comment. This revision is now accepted and ready to land.
Thanks. Sorry I missed this. ================ Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:487-497 { // Switch caller's context with LSPServer's background context. Since we // rather want to propagate information from LSPServer's context into the // Server, CDB, etc. WithContext MainContext(BackgroundContext.clone()); llvm::Optional<WithContextValue> WithOffsetEncoding; if (Opts.Encoding) ---------------- I think we should do this after the Opts has been fully set to avoid the confusion Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133299/new/ https://reviews.llvm.org/D133299 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits