[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