Skip to content

Commit fa6340e

Browse files
committed
Fix build on MacOS (Sequioa)
1 parent 41e5b89 commit fa6340e

2 files changed

Lines changed: 3 additions & 5 deletions

File tree

opensfm/src/CMakeLists.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ endif()
1717

1818
####### Compilation Options #######
1919
# Visibility stuff
20-
cmake_policy(SET CMP0063 NEW)
2120
set(CMAKE_CXX_VISIBILITY_PRESET hidden)
2221
set(CMAKE_CXX_VISIBILITY_INLINES ON)
2322

@@ -39,6 +38,9 @@ add_definitions(-DVL_DISABLE_AVX)
3938

4039
# Use the version of vlfeat in ./src/third_party/vlfeat
4140
add_definitions(-DINPLACE_VLFEAT)
41+
if(NOT USE_SSE2)
42+
add_definitions(-DVL_DISABLE_SSE2)
43+
endif()
4244

4345
if (WIN32)
4446
# Missing math constant

opensfm/src/geometry/triangulation.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -94,11 +94,7 @@ std::pair<bool, Eigen::Matrix<T, 3, 1>> TriangulateTwoBearingsMidpointSolve(
9494

9595
const T eps = T(1e-30);
9696
const T det = A.determinant();
97-
#ifdef __aarch64__
98-
if (std::abs<T>(det) < eps) {
99-
#else
10097
if (abs(det) < eps) {
101-
#endif
10298
return std::make_pair(false, Eigen::Matrix<T, 3, 1>());
10399
}
104100
const auto lambdas = A.inverse() * b;

0 commit comments

Comments
 (0)