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

[Marek Olšák]

Maybe we should split --enable-debug into --enable-assertions and
--enable-optimized. That's what LLVM does.

Marek

On Thu, Jan 10, 2013 at 4:03 AM, Chia-I Wu <olvaffe at gmail.com> wrote:
> 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
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev