Hi Takumi,<div><br></div><div>THANK YOU for your excellent review!  It makes all the work worthwhile in itself.</div><div><br></div><div>It is getting "late" here (it is early morning) so I am about to call it a day.  But I will contemplate and incorporate your suggestions tomorrow, and send you a full reply.  I think I originally wrote a document where I addressed both Visual Studio and MinGW, but then decided that there were way too many "if this" and "if that"s to continue on that path.</div>
<div><br></div><div>Thanks again :-)</div><div><br></div><div><br></div><div>Cheers,</div><div>Mikael</div><div><br></div><div><br><div class="gmail_quote">2012/6/16 NAKAMURA Takumi <span dir="ltr"><<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Mikael,<br>
<br>
Thanks for working on.<br>
<br>
My gross comments...<br>
<br>
- Canonical URL of LLVM project is; <a href="http://llvm.org/" target="_blank">http://llvm.org/</a> , not <a href="http://www.llvm.org" target="_blank">www.llvm.org</a>.<br>
<br>
- "test-suite and cygwin" could be split out. It would be special thing.<br>
<br>
- Memory/storage requirements would be moved to "A.B Debugging". Less<br>
memory would be enough without debug build.<br>
<br>
- Python (x64 binary) is available, too. I am using.<br>
  Python 2.7.1 (r271:86832, Nov 27 2010, 17:19:03) [MSC v.1500 64 bit<br>
(AMD64)] on win32<br>
<br>
- Distinguish "test-suite" and "{llvm|clang} tests". (Installing GnuWin32)<br>
<br>
- Don't recommend to delete MSYS sh.exe in "Installing CMake". Use<br>
MSYS Makefiles instead with msys.<br>
<br>
- gnuwin32 is not needed on %PATH% if gnuwin32 is dedicated to tests.<br>
Use LLVM_LIT_TOOLS_DIR instead then.<br>
  (gnuwin32 is not needed when person is working on MSYS bash shell, FYI)<br>
<br>
- cmake and python are needed on %PATH%, as long as they are not<br>
invoked explicitly.<br>
  cmake-generated Makefiles know where they are.<br>
  eg. "make edit_cache", "make check"<br>
<br>
- If the target of this would be for newbies, you may mention "cmake-gui", too.<br>
  FYI, I am usually use cmake-gui.exe, because I am n00b.<br>
<br>
- (Building the Sources), In my experience, make.exe -jN tends to be<br>
choked. I don't know where is -jN-stable gnu make on mingw. (Lemme<br>
know if you knew better version of make!)<br>
<br>
- Why don't you mention "make {check|clang-test|check-all}", despite<br>
of introducing gnuwin32?<br>
<br>
- You forgot to mention a few important cmake variables.<br>
<br>
  - CMAKE_BUILD_TYPE=Release (for most purpose)<br>
  - CMAKE_INSTALL_PREFIX (Who would be happy if clang were installed<br>
onto C:\PROGRA~1\?)<br>
  - LLVM_LIT_TOOLS_DIR, to use gnuwin32 dedicated to testing llvm and clang.<br>
  - LLVM_EXTERNAL_CLANG_SOURCE_DIR, to check out clang outside from<br>
llvm source tree<br>
  - PYTHON_EXECUTABLE, if %PATH% does not contain python.<br>
<br>
- Consider to introduce ninja?<br>
<br>
Anyway, this could be integrated to "GettingStartedVS.html" ;)<br>
<span><font color="#888888"><br>
...Takumi<br>
</font></span><br>
ps. feel free to visit and ask me on the irc, oftc#llvm.<br>
</blockquote></div><br></div>