Looks good.  Stuff like this feel free to just commit<br><div class="gmail_quote"><div dir="ltr">On Wed, Feb 7, 2018 at 7:53 PM Aaron Smith via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">asmith created this revision.<br>
asmith added reviewers: zturner, lldb-commits.<br>
Herald added a subscriber: llvm-commits.<br>
<br>
LLDB_TEST_COMPILER is not a valid option for CMake for LLDB. There are instead two properties LLDB_TEST_C_COMPILER and LLDB_TEST_CXX_COMPILER. Update the documents accordingly to reflect the correct information.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D43061" rel="noreferrer" target="_blank">https://reviews.llvm.org/D43061</a><br>
<br>
Files:<br>
  www/build.html<br>
  www/test.html<br>
<br>
<br>
Index: www/test.html<br>
===================================================================<br>
--- www/test.html<br>
+++ www/test.html<br>
@@ -39,8 +39,8 @@<br>
               The easiest way to run the LLDB test suite is to use the <tt>check-lldb</tt> build<br>
               target. By default, the <tt>check-lldb</tt> target builds the test programs with<br>
               the same compiler that was used to build LLDB. To build the tests with a different<br>
-              compiler, you can set the <strong>LLDB_TEST_COMPILER</strong> CMake variable. It is possible to<br>
-              customize the architecture of the test binaries and compiler used by appending -A<br>
+              compiler, you can set the <strong>LLDB_TEST_C_COMPILER</strong> or the <strong>LLDB_TEST_CXX_COMPILER</strong> CMake variables.<br>
+              It is possible to customize the architecture of the test binaries and compiler used by appending -A<br>
               and -C options respectively to the CMake variable <strong>LLDB_TEST_USER_ARGS</strong>. For<br>
               example, to test LLDB against 32-bit binaries<br>
               built with a custom version of clang, do:<br>
Index: www/build.html<br>
===================================================================<br>
--- www/build.html<br>
+++ www/build.html<br>
@@ -116,14 +116,14 @@<br>
                 the PYTHONHOME environment variable if it is specified).<br>
               </li><br>
               <li><br>
-                <b>LLDB_TEST_COMPILER</b>: The test suite needs to be able to find a copy of clang.exe that it can use to compile<br>
-                inferior programs.  Note that MSVC is not supported here, it <strong>must</strong> be a path to a clang executable.<br>
-                Note that using a release clang.exe is strongly recommended here, as it will make the test suite run much faster.<br>
+                <b>LLDB_TEST_C_COMPILER</b> or <b>LLDB_TEST_CXX_COMPILER</b>: The test suite needs to be able to find a copy of clang.exe<br>
+                that it can use to compile inferior programs.  Note that MSVC is not supported here, it <strong>must</strong> be a path to a<br>
+                clang executable.  Note that using a release clang.exe is strongly recommended here, as it will make the test suite run much faster.<br>
                 This can be a path to any recent clang.exe, including one you built yourself.<br>
               </li><br>
             </ul><br>
             Sample command line:<br/><br>
-            <code>cmake -G Ninja -DLLDB_TEST_DEBUG_TEST_CRASHES=1 -DPYTHON_HOME=C:\Python35 -DLLDB_TEST_COMPILER=d:\src\llvmbuild\ninja_release\bin\clang.exe ..\..\llvm</code><br>
+            <code>cmake -G Ninja -DLLDB_TEST_DEBUG_TEST_CRASHES=1 -DPYTHON_HOME=C:\Python35 -DLLDB_TEST_C_COMPILER=d:\src\llvmbuild\ninja_release\bin\clang.exe ..\..\llvm</code><br>
             <h2>Working with both Ninja and MSVC</h2><br>
             <p><br>
               Compiling with <code>ninja</code> is both faster and simpler than compiling with MSVC, but chances are you still want<br>
<br>
<br>
</blockquote></div>