On 01/09/2013 01:32 AM, Michel Dänzer wrote:
On Die, 2013-01-08 at 09:31 -0800, Brian Paul wrote:
On 01/08/2013 08:38 AM, Marek Olšák wrote:

Those who do not want optimizations should set the CFLAGS and CXXFLAGS
environment variables. In my opinion, --enable-debug should only set -DDEBUG.

Previously, if we set --enable-debug we got -g -O2 which was pretty
much useless if you planned to use gdb (and why else would you use
--enable-debug?).

I'm using it because src/gallium/auxiliary/util/u_debug.h disables
assertions without DEBUG. Maybe that should be fixed to only disable
them with NDEBUG.

I'm realizing that people want a spectrum of debug vs. optimization options. Personally, I'm only really interested in debug and release modes (-g -O0 -DDEBUG versus -O2) so that's why I wanted --enable-debug to set those flags.

I'm no expert on autoconfig so I'm not sure what the best solution is for this.

Maybe we should all just use CFLAGS to set all our desired flags and remove --enable-debug. Or if --enable-debug is kept, define it to be shorthand for CFLAGS="something".

-Brian
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to