> -----Original Message----- > From: intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx [mailto:intel-gfx- > bounces@xxxxxxxxxxxxxxxxxxxxx] On Behalf Of devon.davies@xxxxxxxxx > Sent: Wednesday, January 08, 2014 4:07 PM > To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Davies, Devon > Subject: [PATCH] build: Added -UNDEBUG to tools/Android.mk > > From: Devon Davies <devon.davies@xxxxxxxxx> > > NDEBUG is defined by default in Android and causes assert(x) to be defined as > void(x). This change ensures that asserts work as expected in tools. > > This flag is already inside tests/Android.mk I discussed and agreed to Devon´s solution with him before, so: Reviewed-by: Oscar Mateo <oscar.mateo@xxxxxxxxx> > Signed-off-by: Devon Davies <devon.davies@xxxxxxxxx> > --- > tools/Android.mk | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/Android.mk b/tools/Android.mk index 4a19b6c..5bda489 > 100644 > --- a/tools/Android.mk > +++ b/tools/Android.mk > @@ -40,7 +40,7 @@ define add_tool > > LOCAL_CFLAGS += -DHAVE_TERMIOS_H > LOCAL_CFLAGS += -DHAVE_STRUCT_SYSINFO_TOTALRAM > - LOCAL_CFLAGS += -DANDROID > + LOCAL_CFLAGS += -DANDROID -UNDEBUG > LOCAL_CFLAGS += -std=c99 > # FIXME: drop once Bionic correctly annotates "noreturn" on pthread_exit > LOCAL_CFLAGS += -Wno-error=return-type > -- > 1.7.9.5 > > --------------------------------------------------------------------- > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx