On Mon, Jan 05, 2015 at 10:33:57AM +0100, Mijo Safradin wrote: > > > >CFLAGS follows OVS_CFLAGS in the compiler command line, and should, so that > >the user can override any automatically determined compiler options. That > >means that the -O0 that the code here added to OVS_CFLAGS didn't really > >have any effect since CFLAGS by default includes "-O2". However, we do > >really want to disable optimizations (because the coverage reports are hard > >to interpret with optimizations), so this commit changes the configure > >script to edit out -O2 from CFLAGS, leaving any other default or > >user-specified options. > > > > The "-O0" got lost, which is needed. Beside this, your patch is much > cleaner. Ack.
Thanks, I applied this to master. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
