Skip to content

Commit

Permalink
[cmake] Target AXV2 ISA when building for AMD64 (#23)
Browse files Browse the repository at this point in the history
* [cmake] Target AXV2 ISA when building for AMD64

Targeting AXV2 is the closest we can get to targeting
Haswell when compiling with msvc. See DEVIN-1142 for
more context.

* Make changes in bcny-firebase.yml instead of CMakeLists.txt

---------

Co-authored-by: kendal <[email protected]>
  • Loading branch information
kendalharland and kendal authored Mar 29, 2024
1 parent b6b3a9f commit a37ec70
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/bcny-firebase.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,8 @@ jobs:
-D FIREBASE_INCLUDE_FIRESTORE=YES `
-D FIREBASE_USE_BORINGSSL=YES `
-D MSVC_RUNTIME_LIBRARY_STATIC=NO `
-D CMAKE_C_FLAGS="/D_HAS_EXCEPTIONS=0 /EHsc-"`
-D CMAKE_CXX_FLAGS="/D_HAS_EXCEPTIONS=0 /EHsc-" `
-D CMAKE_C_FLAGS="/D_HAS_EXCEPTIONS=0 /EHsc- /arch:AVX2"`
-D CMAKE_CXX_FLAGS="/D_HAS_EXCEPTIONS=0 /EHsc- /arch:AVX2" `
-D CMAKE_MSVC_DEBUG_INFORMATION_FORMAT=Embedded `
-D FIREBASE_PYTHON_HOST_EXECUTABLE:FILEPATH=${{ steps.python.outputs.python-path }} `
-D FLATBUFFERS_FLATC_EXECUTABLE=${{ github.workspace }}/BinaryCache/flatbuffers/Release/flatc.exe
Expand Down

0 comments on commit a37ec70

Please sign in to comment.