gazebosim / gz-tools

Command line tools for the Gazebo libraries.

https://gazebosim.org

Apache License 2.0

14 stars 18 forks source link

add x86 check to findSSE.cmake #52

Closed YVbakker closed 3 years ago

YVbakker commented 3 years ago

SSE is only supported on x86 family processors. This disables looking for it on other platforms such as ARM64. Perhaps it would be nicer to skip this check altogether in the root CMakeLists.txt

🦟 Bug fix

Fixes #51

Summary

Adds IF(CMAKE_SYSTEM_PROCESSOR MATCHES "x86_64") clause to CMake/findSSE.cmake to disable looking for SSE on non-x86 hardware systems. Fixes configure error.

Note to maintainers: Remember to use Squash-Merge

j-rivero commented 3 years ago

Thanks for the PR Yael. I've been cleaning up death code in this repo, we probably no longer need this PR after the merge of #55 . Please reopen the PR if I'm wrong.