[PATCH] D27138: Extend CompilationDatabase by a field for the output filename

[Manuel Klimek via Phabricator via cfe-commits]

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