Skip to content

Commit

Permalink
asm trick
Browse files Browse the repository at this point in the history
  • Loading branch information
JonasVautherin committed Feb 8, 2018
1 parent 4125420 commit fcd1a95
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions grpc/server/cmake/third_party/boringssl/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,21 @@ elseif(IOS)
-DCMAKE_INSTALL_PREFIX:PATH=${BINARY_DIR}
-DCMAKE_TOOLCHAIN_FILE:PATH=${CMAKE_TOOLCHAIN_FILE}
-DIOS_PLATFORM=OS
-DOPENSSL_NO_ASM=ON # Note: https://github.com/grpc/grpc/issues/9440
INSTALL_COMMAND true)
else()
# We disable ASM optimizations for MSVC and iOS (see above for iOS), which
# might have a performance impact on secure communications.
# This should be benchmarked/evaluated before we decide if it is worth
# enabling them or not in our case.
#
# More information here: https://github.com/grpc/grpc/issues/9440
if (MSVC)
set(OPENSSL_NO_ASM ON)
else()
set(OPENSSL_NO_ASM OFF)
endif()

message(STATUS "Preparing external project \"boringssl\"...")

ExternalProject_add(
Expand All @@ -50,5 +63,6 @@ else()
CMAKE_ARGS
-DCMAKE_INSTALL_PREFIX:PATH=${BINARY_DIR}
-DCMAKE_POSITION_INDEPENDENT_CODE=ON
-DOPENSSL_NO_ASM=${OPENSSL_NO_ASM}
INSTALL_COMMAND true)
endif()

0 comments on commit fcd1a95

Please sign in to comment.