Re: [ros-users] Calonder descriptor: exporting sse in manife…

Top Page
Attachments:
Message as email
+ (text/plain)
Delete this message
Reply to this message
Author: Radu Bogdan Rusu
Date:  
To: ros-users
Subject: Re: [ros-users] Calonder descriptor: exporting sse in manifest.xml
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
> 
> https://code.ros.org/mailman/listinfo/ros-users


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