[PATCH] D11862: Edits to CMake.rst for clarity's sake

NAKAMURA Takumi via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 22 16:47:00 PDT 2015


chapuni added inline comments.

================
Comment at: docs/CMake.rst:199-201
@@ -190,4 +198,5 @@
   Flag indicating if shared libraries will be built. Its default value is
-  OFF. Shared libraries are not supported on Windows and not recommended on the
-  other OSes.
+  OFF. Shared libraries are supported on Windows when using MinGW, including
+  mingw-w64. They are not supported on Windows when using the Microsoft
+  toolchain. Shared libraries are not recommended on other operating systems.
 
----------------
I wonder it might misunderstand like
"BUILD_SHARED_LIBS is recommended on mingw."

IMO, BUILD_SHARED_LIBS is an option for developers.

================
Comment at: docs/CMake.rst:443
@@ -427,3 +442,3 @@
 
-  $ make check
+  $ make check-all
 
----------------
Yes. in trunk, "check" is an alias of "check-llvm".
For consistency in CMake scripts, "check-*" is the primary target and "check" is compatible stuff for autoconf Make.

You may also mention "check-*" to run tests for individual subproject, here.

================
Comment at: docs/CMake.rst:445
@@ -429,3 +444,3 @@
 
-On Visual Studio, you may run tests to build the project "check".
+On Visual Studio, you may run tests by building the project "check".
 
----------------
Although I am not a user of Xcode, I guess it might be generic for IDE Generators.


http://reviews.llvm.org/D11862





More information about the llvm-commits mailing list