Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1119)

Side by Side Diff: Makefile.standalone

Issue 1311653003: Add UBSAN build option and fix undefined behaviour errors. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | src/IceAssembler.h » ('j') | src/IceInst.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # The following variables will likely need to be modified, depending on where 1 # The following variables will likely need to be modified, depending on where
2 # and how you built LLVM & Clang. They can be overridden in a command-line 2 # and how you built LLVM & Clang. They can be overridden in a command-line
3 # invocation of make, like: 3 # invocation of make, like:
4 # 4 #
5 # make LLVM_SRC_PATH=<path> LIBCXX_INSTALL_PATH=<path> CLANG_PATH=<path> \ 5 # make LLVM_SRC_PATH=<path> LIBCXX_INSTALL_PATH=<path> CLANG_PATH=<path> \
6 # PNACL_BIN_PATH=<path> ... 6 # PNACL_BIN_PATH=<path> ...
7 # 7 #
8 8
9 # LLVM_SRC_PATH is the path to the root of the checked out source code. This 9 # LLVM_SRC_PATH is the path to the root of the checked out source code. This
10 # directory should contain the configure script, the include/ and lib/ 10 # directory should contain the configure script, the include/ and lib/
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 SB_CXX_DEFINES := $(BASE_CXX_DEFINES) -DPNACL_BROWSER_TRANSLATOR=1 89 SB_CXX_DEFINES := $(BASE_CXX_DEFINES) -DPNACL_BROWSER_TRANSLATOR=1
90 CXX_DEFINES := $(BASE_CXX_DEFINES) -DPNACL_BROWSER_TRANSLATOR=0 90 CXX_DEFINES := $(BASE_CXX_DEFINES) -DPNACL_BROWSER_TRANSLATOR=0
91 91
92 ifdef NOASSERT 92 ifdef NOASSERT
93 ASSERTIONS = -DNDEBUG 93 ASSERTIONS = -DNDEBUG
94 else 94 else
95 ASSERTIONS = 95 ASSERTIONS =
96 OBJDIR := $(OBJDIR)+Asserts 96 OBJDIR := $(OBJDIR)+Asserts
97 endif 97 endif
98 98
99 ifdef UBSAN
100 OBJDIR := $(OBJDIR)+UBSan
101 CXX_EXTRA += -fsanitize=undefined -fno-sanitize=vptr
102 LD_EXTRA += -fsanitize=undefined
103 endif
104
99 ifdef TSAN 105 ifdef TSAN
100 OBJDIR := $(OBJDIR)+TSan 106 OBJDIR := $(OBJDIR)+TSan
101 CXX_EXTRA += -fsanitize=thread 107 CXX_EXTRA += -fsanitize=thread
102 LD_EXTRA += -fsanitize=thread 108 LD_EXTRA += -fsanitize=thread
103 endif 109 endif
104 110
105 ifdef ASAN 111 ifdef ASAN
106 OBJDIR := $(OBJDIR)+ASan 112 OBJDIR := $(OBJDIR)+ASan
107 CXX_EXTRA += -fsanitize=address 113 CXX_EXTRA += -fsanitize=address
108 LD_EXTRA += -fsanitize=address 114 LD_EXTRA += -fsanitize=address
(...skipping 289 matching lines...) Expand 10 before | Expand all | Expand 10 after
398 404
399 docs: 405 docs:
400 doxygen Doxyfile 406 doxygen Doxyfile
401 @echo See file://`pwd`/docs/html/index.html 407 @echo See file://`pwd`/docs/html/index.html
402 408
403 clean: 409 clean:
404 rm -rf pnacl-sz *.o $(OBJDIR) $(SB_OBJDIR) build/pnacl-sz.bloat.json 410 rm -rf pnacl-sz *.o $(OBJDIR) $(SB_OBJDIR) build/pnacl-sz.bloat.json
405 411
406 clean-all: clean 412 clean-all: clean
407 rm -rf build/ docs/ 413 rm -rf build/ docs/
OLDNEW
« no previous file with comments | « no previous file | src/IceAssembler.h » ('j') | src/IceInst.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698