follow comments

Author: typhoonzero

CMakeLists.txt

@@ -103,6 +103,11 @@ if(ANDROID OR IOS)
     add_definitions(-DPADDLE_MOBILE_INFERENCE)
 endif()
 
+if (APPLE)
+    set(WITH_MKL OFF CACHE STRING
+        "Disable MKL for building on mac" FORCE)
+endif()
+
 set(THIRD_PARTY_PATH "${CMAKE_BINARY_DIR}/third_party" CACHE STRING
   "A path setting third party libraries download & build directories.")
 

cmake/external/mkldnn.cmake

@@ -28,9 +28,6 @@ IF(WIN32 OR APPLE)
         "Windows or Mac is not supported with MKLDNN in Paddle yet."
         "Force WITH_MKLDNN=OFF")
     SET(WITH_MKLDNN OFF CACHE STRING "Disable MKLDNN in Windows and MacOS" FORCE)
-    IF ((NOT ${WITH_MKLDNN}) AND (NOT ${WITH_MKLML}))
-        SET(WITH_MKL OFF CACHE STRING "Disable MKL for later scripts" FORCE)
-    ENDIF()
     return()
 ENDIF()
 

cmake/external/mklml.cmake

@@ -21,9 +21,6 @@ IF(WIN32 OR APPLE)
         "Windows or Mac is not supported with MKLML in Paddle yet."
         "Force WITH_MKLML=OFF")
     SET(WITH_MKLML OFF CACHE STRING "Disable MKLML package in Windows and MacOS" FORCE)
-    IF ((NOT ${WITH_MKLDNN}) AND (NOT ${WITH_MKLML}))
-        SET(WITH_MKL OFF CACHE STRING "Disable MKL for later scripts" FORCE)
-    ENDIF()
     return()
 ENDIF()