klimek added inline comments.
================ Comment at: lib/Tooling/JSONCompilationDatabase.cpp:266 + nodeToCommandLine(Syntax, std::get<2>(CommandsRef[I])), + Output ? Output->getValue(OutputStorage) : ""); } ---------------- joerg wrote: > klimek wrote: > > Optional: I'd probably let the nodeToCommandLine handle the null value and > > make this code more straight forward? > I couldn't find a way to create a synthetic node without changing the YAML > API. I'm probably missing something - why would we need a synthetic node? Can't we just put nullptr into the vector? Repository: rL LLVM https://reviews.llvm.org/D27138 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
