[llvm-commits] CVS: llvm/lib/CodeGen/SelectionDAG/SelectionDAGPrinter.cpp

[Chris Lattner]

On Wed, 13 Jul 2005, Reid Spencer wrote:
>>> None of this code is portable because of the use of the system call. I
>>> know its convenient but windows doesn't have it. We have a facility in
>>> lib/Support for executing external programs. You should use it.
>>
>> I'm not convinced.  Windows won't have 'gv' or 'Graphviz' either.  I'd be
>> happy to make the second one depend on gv being found by autoconf though.
>
> That's crap, Chris. Here's the Graphviz Win32 download page:
>
> http://home.so-net.net.tw/oodtsen/wingraphviz/index.htm
>
> Graphviz is definitely available on Windows.  Windows users already
> suffer enough with LLVM, why add further deprecated features?

FWIW, as discussed on IRC, "Graphviz" the package and "Graphviz" the 
program are not the same thing.  Graphviz the program is an OS/X specific 
thing.

> As for gv, we should have a test and a path variable for it too ..
> actually for *all* programs we rely on. I'll add gv. Let me know if
> there are others.

Totally agreed, thanks!

-Chris

-- 
http://nondot.org/sabre/
http://llvm.org/