Re: [PATCH] D10365: Add cmd to compilation database file format

2015-08-13 Thread Daniel Dilts via cfe-commits
diltsman updated this revision to Diff 32082. diltsman marked an inline comment as done. diltsman added a comment. Arguments and Command can now be in the same compilation database for the same file. Arguments are preferred when both are present. http://reviews.llvm.org/D10365 Files:

Re: [PATCH] D10365: Add cmd to compilation database file format

2015-08-13 Thread Daniel Dilts via cfe-commits
diltsman updated this revision to Diff 32099. diltsman marked an inline comment as done. diltsman added a comment. Added test for command and arguments in same object. Fixed bug where parse failed if command and argument resolved to empty lists. http://reviews.llvm.org/D10365 Files:

Re: [PATCH] D10365: Add cmd to compilation database file format

2015-08-12 Thread Daniel Dilts via cfe-commits
diltsman updated this revision to Diff 31952. diltsman marked an inline comment as done. http://reviews.llvm.org/D10365 Files: ../llvm/tools/clang/include/clang/Tooling/JSONCompilationDatabase.h ../llvm/tools/clang/lib/Tooling/JSONCompilationDatabase.cpp