[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