Skip to content

Commit

Permalink
Merge pull request #159 from DUNE/kleykamp_make_sanitize
Browse files Browse the repository at this point in the history
Add 'make sanitize' which is super useful. Super. Useful.
  • Loading branch information
jdkio authored Sep 25, 2024
2 parents a3fa6b7 + 437c149 commit 8ce130c
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 0 deletions.
4 changes: 4 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,3 +18,7 @@ clean:
all: check-submodule
make -j8 -C src
make -j8 -C app

sanitize: check-submodule
make -j8 -C src sanitize
make -j8 -C app sanitize
4 changes: 4 additions & 0 deletions app/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,10 @@ MKDIR_P := mkdir -p

all: directories $(EXE_DIR)/ConvertToTMSTree.exe $(EXE_DIR)/BetheBloch_Example.exe $(EXE_DIR)/DBSCAN_test.exe $(EXE_DIR)/DrawEvents.exe $(EXE_DIR)/TrackLengthTester.exe $(EXE_DIR)/ShootRay.exe $(EXE_DIR)/BField_tester.exe $(EXE_DIR)/CherryPickEvents.exe

# Sanitize target
sanitize: CXXFLAGS += -fsanitize=address
sanitize: all

directories: $(EXE_DIR)

$(EXE_DIR):
Expand Down
4 changes: 4 additions & 0 deletions src/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,10 @@ endif

all: directories libTMS_Prod

# Sanitize target
sanitize: CXXFLAGS := $(CXXFLAGS) -fsanitize=address
sanitize: all

directories: $(LIB_DIR)

$(LIB_DIR):
Expand Down

0 comments on commit 8ce130c

Please sign in to comment.