This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2e1bb7940a4d: [clangd] Add missing "override" to fix the build. (authored by adamcz).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94314/new/ https://reviews.llvm.org/D94314 Files: clang-tools-extra/clangd/AST.cpp Index: clang-tools-extra/clangd/AST.cpp =================================================================== --- clang-tools-extra/clangd/AST.cpp +++ clang-tools-extra/clangd/AST.cpp @@ -310,7 +310,7 @@ public: PrintCB(const DeclContext *CurContext) : CurContext(CurContext) {} virtual ~PrintCB() {} - virtual bool isScopeVisible(const DeclContext *DC) const { + virtual bool isScopeVisible(const DeclContext *DC) const override { return DC->Encloses(CurContext); }
Index: clang-tools-extra/clangd/AST.cpp =================================================================== --- clang-tools-extra/clangd/AST.cpp +++ clang-tools-extra/clangd/AST.cpp @@ -310,7 +310,7 @@ public: PrintCB(const DeclContext *CurContext) : CurContext(CurContext) {} virtual ~PrintCB() {} - virtual bool isScopeVisible(const DeclContext *DC) const { + virtual bool isScopeVisible(const DeclContext *DC) const override { return DC->Encloses(CurContext); }
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits