[PATCH] D23823: [builtins] Make sure builtin compile tests respect CMAKE_C_COMPILER_TARGET

Saleem Abdulrasool via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 23 21:54:48 PDT 2016


compnerd accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: cmake/Modules/BuiltinTests.cmake:17
@@ +16,3 @@
+  if(CMAKE_C_COMPILER_TARGET)
+    list(APPEND TRY_COMPILE_FLAGS "-target ${CMAKE_C_COMPILER_TARGET}")
+  endif()
----------------
I think I prefer the following syntax:

    list(APPEND TRY_COMPILE_FLAGS -target;${CMAKE_C_COMPILER_TARGET})


https://reviews.llvm.org/D23823





More information about the llvm-commits mailing list