[PATCH] D92640: Add ability to load a FixedCompilationDatabase from a buffer.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 4 13:09:14 PST 2020


sammccall marked an inline comment as done.
sammccall added inline comments.


================
Comment at: clang/include/clang/Tooling/CompilationDatabase.h:194
+  static std::unique_ptr<FixedCompilationDatabase>
+  loadFromBuffer(StringRef Path, StringRef Data, std::string &ErrorMsg);
+
----------------
usaxena95 wrote:
> usaxena95 wrote:
> > `Path` is not meaningful for load from buffer. May be change name to `Directory` and call loadFromBuffer(**llvm::sys::path::parent_path(Path)**, (*File)->getBuffer(), ErrorMsg); instead.
> Do you intend to use it apart from testing ?
Yes. See D92663


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92640/new/

https://reviews.llvm.org/D92640



More information about the cfe-commits mailing list