[pisa-src] Re: r2298 - in trunk: configure.ac libpisa/arp.c libpisa/debug.h libpisa/global.h libpisa/hitlist.c libpisa/nat.c libpisa/sysdep_fw.c libpisa/sysdep_tun.c pairing/libconfig_wrapper.c pairing/packet_han...
- From: Diego Biurrun <diego@xxxxxxxxxx>
- To: pisa-src@xxxxxxxxxxxxx
- Date: Mon, 17 May 2010 15:31:09 +0200
On Mon, May 17, 2010 at 12:39:13PM +0200, Mircea Gherzan wrote:
> Diego Biurrun <diego@xxxxxxxxxx> wrote:
> > > +if test "$(uname -s)" = "FreeBSD"; then
> > > + AC_SUBST(AM_CFLAGS, "$AM_CFLAGS -fno-strict-aliasing")
> > > +fi
> >
> > What's the effect of this flag? I notice that HIPL has it as well,
> > why not add it to the global CFLAGS?
>
> Strict aliasing [1] enables further optimizations (think reordering) from the
> compiler and is enabled by default starting from -O2. So the flag above is not
> should not be used for global CFALGS.
Then why is it necessary for FreeBSD? I'm getting the impression that
there is an underlying problem in need of fixing.
Probably we should also try to remove the flag from HIPL...
Diego
Other related posts:
