Skip to content

Commit

Permalink
build: do not add sse flags when doing MacOS arm cross build (VowpalW…
Browse files Browse the repository at this point in the history
…abbit#4235)

* build: do not add sse flags when doing MacOS arm cross build

* fix var

* fix

* try new fix
  • Loading branch information
jackgerrits authored Oct 20, 2022
1 parent 7434088 commit 1666d3d
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion cmake/VWFlags.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,9 @@ if(LTO)
endif()

if("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "x86_64")
set(LINUX_X86_64_OPT_FLAGS -msse2 -mfpmath=sse)
if(NOT "arm64" STREQUAL "${CMAKE_OSX_ARCHITECTURES}")
set(LINUX_X86_64_OPT_FLAGS -msse2 -mfpmath=sse)
endif()
endif()

# Add -ffast-math for speed, remove for testability.
Expand Down

0 comments on commit 1666d3d

Please sign in to comment.