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

[Chia-I Wu]

On Wed, Jan 9, 2013 at 5:32 PM, Michel Dänzer <michel at daenzer.net> 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 agree with Michel here.  I was surprised to find out that assert()
is compiled to no-op in gallium without --enable-debug.

>
> --
> Earthling Michel Dänzer           |                   http://www.amd.com
> Libre software enthusiast         |          Debian, X and DRI developer
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev



-- 
olv at LunarG.com