Added sanitizer flags to Makevars.in #52

Author: gregorgorjanc

RcppTskit/src/Makevars.in

@@ -30,7 +30,7 @@ RCPPTSKIT_OBJECTS = $(RCPPTSKIT_C_SOURCES:.c=.o) $(RCPPTSKIT_CPP_SOURCES:.cpp=.o
 OBJECTS = $(TSKIT_OBJECTS) $(RCPPTSKIT_OBJECTS)
 # $(info OBJECTS = $(OBJECTS)) # for debugging, info is also a GNU extension
 
-# Include paths and preprocessor defines
+# *Preprocessor (CPP)* flags for include paths and defines
 # * A bit complicated include/tskit structure due to how we include tskit headers
 # * $(R_INCLUDE_DIR) is for <R_ext/Error.h>
 PKG_CPPFLAGS = \
@@ -39,14 +39,25 @@ PKG_CPPFLAGS = \
 	-I../inst/include/tskit/tskit \
 	-I$(R_INCLUDE_DIR)
 
-# Compiler flags
-PKG_CFLAGS = -DNDEBUG # to remove calls to assert() as per the R extensions manual
-# PKG_CFLAGS = -DNDEBUG -DTSK_TRACE_ERRORS # to also enable error tracing in tskit C
-# PKG_CXXFLAGS = -DTSK_TRACE_ERRORS # to also enable error tracing in tskit C as called from Rcpp
+# *Compiler (C/CXX)* flags
+# Uncomment for local debugging
+# TSK_TRACE_ERRORS enables tskit C error tracing
+SAN_FLAGS = -g -O1 -fno-omit-frame-pointer -fsanitize=address,undefined -fno-common
+RCPPTSKIT_CFLAGS = $(SAN_FLAGS) -DTSK_TRACE_ERRORS
+RCPPTSKIT_CXXFLAGS = $(SAN_FLAGS) -DTSK_TRACE_ERRORS
+RCPPTSKIT_LDFLAGS = -fsanitize=address,undefined
+# Uncomment for release builds
+# NDEBUG removes calls to assert()
+# RCPPTSKIT_CFLAGS = -DNDEBUG
+# RCPPTSKIT_CXXFLAGS = -DNDEBUG
+# RCPPTSKIT_LDFLAGS =
+# Use the above choices
+PKG_CFLAGS = $(RCPPTSKIT_CFLAGS)
+PKG_CXXFLAGS = $(RCPPTSKIT_CXXFLAGS)
 
 # Explicit compile rule for tskit C files
 tskit/%.o: tskit/%.c
 	$(CC) $(ALL_CPPFLAGS) $(PKG_CFLAGS) $(CFLAGS) $(CPICFLAGS) -c $< -o $@
 
 # Linking
-PKG_LIBS = @RCPPTSKIT_LIB@
+PKG_LIBS = @RCPPTSKIT_LIB@ $(RCPPTSKIT_LDFLAGS)