[PATCH] D40439: [Tooling] Remove file/command enumeration from CompilationDatabase.

2017-11-28 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. In https://reviews.llvm.org/D40439#935678, @bkramer wrote: > There are a few users of the C++ API out there, do we have migration path for > them? Do we have any idea what these look like? One option would be to keep the functionality on the JSONCompilationDatabase

[PATCH] D40439: [Tooling] Remove file/command enumeration from CompilationDatabase.

2017-11-27 Thread Benjamin Kramer via Phabricator via cfe-commits
bkramer added a comment. There are a few users of the C++ API out there, do we have migration path for them? https://reviews.llvm.org/D40439 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-

[PATCH] D40439: [Tooling] Remove file/command enumeration from CompilationDatabase.

2017-11-24 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. Herald added a subscriber: klimek. In practice, these methods aren't used, and several databases don't implement them. They remain in libclang as a stub for binary compatibility. c-index-tests are modified not to rely on them. https://reviews.llvm.org/D40439 Fil