[PATCH] D64541: rL365634 adds a unique_ptr<CompilationDatabase> in GobalCompilationDatabase.h:108 but CompilationDatabase is only forward declared. This makes the header not compile standalone, because unique_ptrs expect to have the full-definition of the...
George Burgess IV via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 10 16:25:45 PDT 2019
george.burgess.iv accepted this revision.
george.burgess.iv added a comment.
This revision is now accepted and ready to land.
LGTM with one change -- thanks!
================
Comment at: clang-tools-extra/clangd/GlobalCompilationDatabase.h:24
-class CompilationDatabase;
struct CompileCommand;
} // namespace tooling
----------------
looks like clang/Tooling/CompilationDatabase.h has this, too; please delete
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64541/new/
https://reviews.llvm.org/D64541
More information about the llvm-commits
mailing list