[ros-users] Calonder descriptor: exporting sse in manifest.xml

[Radu Bogdan Rusu]

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
> <markus.achtelik at mavt.ethz.ch>  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 at code.ros.org
> https://code.ros.org/mailman/listinfo/ros-users

-- 
| Radu Bogdan Rusu | http://rbrusu.com/