[PATCH] D52636: GraphWriter: Provide an API for writing a graph into a specified file

Artem Dergachev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 28 11:07:48 PDT 2018


NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.

I highly appreciate, because i'm dumping graphs every day, and this looks very handy.



================
Comment at: llvm/include/llvm/Support/GraphWriter.h:339
+      llvm::errs() << "error writing into file" << "\n";
+      return "";
+    }
----------------
This function returns `Filename` on the success branch, should we pass it through here as well?


https://reviews.llvm.org/D52636





More information about the llvm-commits mailing list