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 <olva...@gmail.com> wrote: > On Wed, Jan 9, 2013 at 5:32 PM, Michel Dänzer <mic...@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@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/mesa-dev > > > > -- > o...@lunarg.com > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
