[PATCH] Make compiler-rt tests work with relocatable SDKs on OS X

Alexander Potapenko glider at google.com
Tue Jun 10 08:32:35 PDT 2014


Looks mostly good (see the comments). I'm not sure this doesn't break on 10.8, but let's see if any problems appear.

================
Comment at: compiler-rt/trunk/cmake/Modules/AddCompilerRT.cmake:143
@@ -142,1 +142,3 @@
 
+# Unit tests on Mac depend on Foundation.
+if(APPLE)
----------------
This comment is misleading. This build config is for compiler-rt, not unit tests.

================
Comment at: compiler-rt/trunk/lib/asan/tests/CMakeLists.txt:68
@@ -67,3 @@
-if(APPLE)
-  list(APPEND ASAN_UNITTEST_COMMON_LINKFLAGS -framework Foundation)
-endif()
----------------
Why remove this line? The tests actually do depend on Foundation.

http://reviews.llvm.org/D4047






More information about the llvm-commits mailing list