[Mesa-dev] [PATCH] Revert "configure.ac: Disable compiler optimizations when --enable-debug is set"

[Brian Paul]

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