arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. I think you have to test the -p option too (check the compilation database can be loaded and arguments are propagated to the parser), otherwise LGTM! https://reviews.llvm.org/D36177