add -DNDEBUG for non-debug build (#1006)

* add -DNDEBUG for non-debug build * Don't define NDEBUG if USE_ASSERTION is TRUE
AMReX-Codes · Jun 11, 2020 · c69e431 · c69e431
1 parent d73c9d1
commit c69e431
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/Tools/GNUMake/Make.defs b/Tools/GNUMake/Make.defs
@@ -825,6 +825,12 @@ ifeq ($(AMREX_XSDK),TRUE)
    DEFINES += -DAMREX_XSDK
 endif
 
+ifneq ($(DEBUG),TRUE)
+ifneq ($(USE_ASSERTION),TRUE)
+  DEFINES += -DNDEBUG
+endif
+endif
+
 CPPFLAGS	+= $(DEFINES)
 
 libraries	= $(XTRAOBJS) $(LIBRARIES) $(XTRALIBS)

diff --git a/Tools/GNUMake/Make.rules b/Tools/GNUMake/Make.rules
@@ -448,6 +448,7 @@ ifneq ($(MAKECMDGOALS),clean)
 ifneq ($(MAKECMDGOALS),uninstall)
 ifneq ($(MAKECMDGOALS),distclean)
 ifneq ($(findstring print-,$(MAKECMDGOALS)),print-)
+ifneq ($(MAKECMDGOALS),help)
 
 ifeq ($(USE_GPU_PRAGMA),TRUE)
 include $(TmpBuildDir)/$(optionsSuffix)_cuda_done
@@ -478,6 +479,7 @@ endif
 endif
 endif
 endif
+endif
 
 ifneq ($(MAKECMDGOALS),$(filter $(MAKECMDGOALS),clean realclean file_locations tags TAGS uninstall distclean))
   ifdef has_compiler_mismatch