From bac4265a0b9f8e8410d3c898259a412665d7b349 Mon Sep 17 00:00:00 2001
From: helixhorned <helixhorned@1a8010ca-5511-0410-912e-c29ae57300e0>
Date: Sun, 18 Aug 2013 19:24:15 +0000
Subject: [PATCH] For Clang debugging build, only pass selected UBSanitizer
 options.

This makes CLANG=1 RELEASE=0 DEBUGANYWAY=2 OPTLEVEL=1 builds definitely pretty
playable (only Polymost tested) while having a decent amount of sanitization.

git-svn-id: https://svn.eduke32.com/eduke32@4020 1a8010ca-5511-0410-912e-c29ae57300e0
---
 polymer/eduke32/Makefile.common | 13 ++++++-------
 polymer/eduke32/build/Makefile  |  3 +--
 2 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/polymer/eduke32/Makefile.common b/polymer/eduke32/Makefile.common
index 0076d2cde..1cbe7a9b4 100644
--- a/polymer/eduke32/Makefile.common
+++ b/polymer/eduke32/Makefile.common
@@ -335,6 +335,11 @@ endif
 
 debug+= -O$(OPTLEVEL)
 
+# See http://clang.llvm.org/docs/UsersManual.html#controlling-code-generation
+# for a list of possible UBSan options.
+# Clang 3.2 does only supports -fsanitize=address for the AddressSanitizer
+CLANG_DEBUG_FLAGS := -fsanitize=address -fsanitize=bounds,enum,float-cast-overflow,object-size
+
 ifneq (0,$(RELEASE))
     # Debugging disabled
     ifeq (0,$(CLANG))
@@ -366,13 +371,7 @@ else
     endif
 
     ifneq (0,$(CLANG))
-        # KEEPINSYNC with build/Makefile:
-        # Older Clang (2.9?)
-#        BASECFLAGS+= -Xclang -fcatch-undefined-behavior
-        # Clang 3.2
-#        debug+= -fsanitize=address
-        # Clang 3.3 or SVN
-        debug+= -fsanitize=address -fsanitize=undefined
+        debug+= $(CLANG_DEBUG_FLAGS)
     endif
     ifeq ($(SUBPLATFORM),LINUX)
         BASELIBS+=-rdynamic
diff --git a/polymer/eduke32/build/Makefile b/polymer/eduke32/build/Makefile
index c3ef61cc5..1ac10a1ad 100644
--- a/polymer/eduke32/build/Makefile
+++ b/polymer/eduke32/build/Makefile
@@ -291,8 +291,7 @@ $(OBJ)/%.$o: $(SRC)/%.nasm
 # Comment out the following rule to debug a-c.o
 $(OBJ)/a-c.$o: $(SRC)/a-c.c
 	$(COMPILE_STATUS)
-	if $(COMPILER) $(subst -O$(OPTLEVEL),-O2,$(subst -fsanitize=address -fsanitize=undefined,,$(OURCFLAGS))) -c $< -o $@; then $(COMPILE_OK); else $(COMPILE_FAILED); fi
-# KEEPINSYNC the Clang -f option(s) with Makefile.common
+	if $(COMPILER) $(subst -O$(OPTLEVEL),-O2,$(subst $(CLANG_DEBUG_FLAGS),,$(OURCFLAGS))) -c $< -o $@; then $(COMPILE_OK); else $(COMPILE_FAILED); fi
 
 $(OBJ)/%.$o: $(SRC)/%.c
 	$(COMPILE_STATUS)