joerg added inline comments.

================
Comment at: lib/Tooling/JSONCompilationDatabase.cpp:266
+        nodeToCommandLine(Syntax, std::get<2>(CommandsRef[I])),
+        Output ? Output->getValue(OutputStorage) : "");
   }
----------------
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.


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

Reply via email to