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

diltsman added a comment. Yes, please land it. http://reviews.llvm.org/D10365 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

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: ../llvm/t

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

diltsman added a comment. Where/how does documentation (http://clang.llvm.org/docs/JSONCompilationDatabase.html) get updated? http://reviews.llvm.org/D10365 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailm

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

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: ../llv

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

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 ../llvm/tools/clang/unittests/Toolin

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

diltsman added inline comments. Comment at: ../llvm/tools/clang/lib/Tooling/JSONCompilationDatabase.cpp:299 @@ +298,3 @@ +if (CommandFound) { + ErrorMessage = "Multiple command and arguments found"; + return false; klimek wrote: > Any rea

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

diltsman updated this revision to Diff 31824. http://reviews.llvm.org/D10365 Files: ../llvm/tools/clang/include/clang/Tooling/JSONCompilationDatabase.h ../llvm/tools/clang/lib/Tooling/JSONCompilationDatabase.cpp ../llvm/tools/clang/unittests/Tooling/CompilationDatabaseTest.cpp Index: ../ll

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

diltsman added a subscriber: diltsman. diltsman added a comment. I'm trying to upload a new patch and I get an error from Phabricator: 'Unhandled Exception ("Exception")'. Any ideas how to get this to work? I have attached the diff file. - F722854: Arguments.patch