This revision was automatically updated to reflect the committed changes. hokein marked an inline comment as done. Closed by commit rCTE334176: [clangd] Fix using the incorrect Index for go-to-definition. (authored by hokein, committed by ).
Changed prior to commit: https://reviews.llvm.org/D47869?vs=150276&id=150279#toc Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D47869 Files: clangd/ClangdServer.cpp Index: clangd/ClangdServer.cpp =================================================================== --- clangd/ClangdServer.cpp +++ clangd/ClangdServer.cpp @@ -302,7 +302,7 @@ llvm::Expected<InputsAndAST> InpAST) { if (!InpAST) return CB(InpAST.takeError()); - CB(clangd::findDefinitions(InpAST->AST, Pos, this->FileIdx.get())); + CB(clangd::findDefinitions(InpAST->AST, Pos, Index)); }; WorkScheduler.runWithAST("Definitions", File, Bind(Action, std::move(CB)));
Index: clangd/ClangdServer.cpp =================================================================== --- clangd/ClangdServer.cpp +++ clangd/ClangdServer.cpp @@ -302,7 +302,7 @@ llvm::Expected<InputsAndAST> InpAST) { if (!InpAST) return CB(InpAST.takeError()); - CB(clangd::findDefinitions(InpAST->AST, Pos, this->FileIdx.get())); + CB(clangd::findDefinitions(InpAST->AST, Pos, Index)); }; WorkScheduler.runWithAST("Definitions", File, Bind(Action, std::move(CB)));
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits