basic_math requires just SSE2, so adding -msse2 should be enough. On 05/12/2010 08:14 AM, Brian Gerkey wrote: > On Wed, May 12, 2010 at 12:54 AM, Markus Achtelik > wrote: >> I was trying to compile own code using the calonder descriptors. I got >> an error when compiling basic_math.h saying that sse is not enabled. >> Adding "-msse3" to the compile flags in my CMakelists.txt or exporting >> it in the calonder_descriptor's manifest.xml solved the problem. Would >> it make sense to export sse by default in the manifest.xml? > > hi Markus, > > Looking at features_2d, which depends on calonder_descriptor, I see > this in its CMakeLists.txt: > # TODO Shouldn't require SSE, but needed to compile against > calonder_descriptor > add_definitions(-Wall -O3 -mfpmath=sse -mmmx -msse3) > In the stereo_utils package, which also depends on > calonder_descriptor, something similar is done, on a target-by-target > basis, e.g.: > rosbuild_add_compile_flags(stereo_utils_lowlevel_py -g > -Wno-missing-field-initializers -msse3) > So our code is already working around the fact that > calonder_descriptor requires, but does not export SSE flags. > > Is it possible to restructure libcalonder so that users of the library > don't need to know about its SSE usage? > > If not, can someone suggest which flags should be hardcoded into > calonder_descriptor's manifest (e.g., "-mfpmath=sse -mmmx -msse3" vs. > just "-msse3")? > > brian. > _______________________________________________ > ros-users mailing list > ros-users@code.ros.org > https://code.ros.org/mailman/listinfo/ros-users -- | Radu Bogdan Rusu | http://rbrusu.com/