Re: [Discuss-gnuradio] More On Ticket 141

[Eric Blossom]

On Mon, Mar 05, 2007 at 10:26:20PM -0500, Michael Dickens wrote:
> Got it!  It's the line:
> 
> swig_CXXFLAGS="-g1 -O1"
> 
> in "gnuradio/configure.ac".  Changing this to "-O2" does the trick.   
> I've created a branch at < http://gnuradio.org/svn/gnuradio/branches/ 
> developers/michaelld/t141 > with the change.  I don't think it will  
> affect any other os except Darwin ... hope I got the syntax correct.   
> It works for me.  Here's the svn diff:
> 
> Index: configure.ac
> ===================================================================
> --- configure.ac        (revision 4720)
> +++ configure.ac        (working copy)
> @@ -49,7 +49,15 @@
>    autoconf_default_CXXFLAGS="$CXXFLAGS"
>    CXXFLAGS=""
>    if test "$GXX" = yes; then
> -    swig_CXXFLAGS="-g1 -O1"
> +    case "$host_os" in
> +       darwin*)
> +       # "-O1" breaks PPC-OSX for some reason
> +       swig_CXXFLAGS="-g -O2"
> +       ;;
> +       *)
> +       swig_CXXFLAGS="-g1 -O1"
> +       ;;
> +    esac
>    fi
> fi
> AC_SUBST(autoconf_default_CXXFLAGS)
> 

Michael, this is great info.

you made two changes:

  -g1 -> -g
  -O1 -> -O2

Does -g -O1 work on PPC-OSX?

We really want to "turn down" the level of optimization used while
compiling the SWIG stuff.  The SWIG stuff is not performance critical,
and using a lower level of optimization speeds up the compilation.

Eric