[llvm-commits] [llvm] r114167 - /llvm/trunk/docs/GettingStartedVS.html

[Óscar Fuentes]

Hello Michael. Thanks for updating the information about testing and
replacing llvm-gcc with clang. Some comments follow.

"Michael J. Spencer"
<bigcheesegs at gmail.com> writes:

[snip]

> +  <p>Welcome to LLVM on Windows! This document only covers native Windows, not
> +  mingw or cygwin.

MinGW is "native" by any definition of the word.

> [+] In order to get started, you first need to know some basic
> +  information.</p>

This is not in line with the "read the documentation twice and then do
it again" advice of GettingStarted.html. This document is not for
learning what LLVM is, but for putting it to work.

[snip]

> +  <p>Another useful project on Windows is
> +  <a href="http://clang.llvm.org/">clang</a>. Clang is a C family
> +  ([Objective]C/C++) compiler. Clang fully works on Windows, but does not
> +  currently understand all of the Microsoft extensions to C and C++. Because of
> +  this, clang cannot parse the C++ standard library included with Visual Studio,
> +  nor parts of the Windows Platform SDK.

A C++ compiler that can't compile programs that use the SC++L or the
platform SDK hardly deserves the "fully works" tag, IMO.

[snip]

> +  <p>The LLVM test suite cannot be run on the Visual Studio port at this
> +  time.</p>

Apparently, you contradict yourself a few lines below by giving
instructions on how to run the tests. You should mention that the LLVM
test suite is not the same as the tests included with LLVM and Clang:

[snip]

> +  <p>If you would like to run the LLVM tests you will need
> +  <a href="http://www.python.org/">Python</a>. Versions 2.4-2.7 are known to
> +  work.</p>

[snip]

> +  <li>Test LLVM:
> +  <ul>
> +    <li>The LLVM tests can be run by <tt>cd</tt>ing to the llvm source directory
> +        and running:

[snip]