[PATCH] D23656: [CMake] Fix ASM building in llvm/runtimes
Chris Bieneman via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 18 13:54:38 PDT 2016
beanz added inline comments.
================
Comment at: CMakeLists.txt:12
@@ +11,3 @@
+# runtimes directory.
+if (CMAKE_SOURCE_DIR STREQUAL CMAKE_CURRENT_SOURCE_DIR OR
+ "${CMAKE_SOURCE_DIR}/compiler-rt" STREQUAL CMAKE_CURRENT_SOURCE_DIR)
----------------
Actually, I do have one suggestion here. Make this:
```
if (CMAKE_SOURCE_DIR STREQUAL CMAKE_CURRENT_SOURCE_DIR OR COMPILER_RT_STANDALONE_BUILD)
```
The runtimes subdirectory CMake file sets the *_STANDALONE_BUILD variable, so we can use it before it would be set in the normal standalone build.
https://reviews.llvm.org/D23656
More information about the llvm-commits
mailing list