[llvm-commits] [llvm] r114832 - in /llvm/trunk: autoconf/configure.ac configure include/llvm/Config/config.h.in lib/Support/GraphWriter.cpp

[Chris Lattner]

On Sep 27, 2010, at 9:28 AM, Dan Gohman wrote:

> +++ llvm/trunk/lib/Support/GraphWriter.cpp Mon Sep 27 11:28:34 2010
> @@ -69,6 +69,32 @@
>   else
>     Filename.eraseFromDisk();
> 
> +#elif HAVE_XDOT_PY
> +  sys::Path XDotPy();

Does this work?  Aren't you declaring a function here, not default constructing a sys::Path?

-Chris

> +
> +  std::vector<const char*> args;
> +  args.push_back(LLVM_PATH_XDOT_PY);
> +  args.push_back(Filename.c_str());
> +
> +  switch (program) {
> +  case GraphProgram::DOT:   args.push_back("-f"); args.push_back("dot"); break;