release build treat warnings as errors

Summary:
fixing warnings is important, especially for release code.
Closes https://github.com/facebook/rocksdb/pull/2971

Differential Revision: D5980596

Pulled By: ajkr

fbshipit-source-id: 04f4ea3fb005dcda33d60342e4361e380bc4dfb1
This commit is contained in:
Andrew Kryczka 2017-10-05 12:32:55 -07:00 committed by Facebook Github Bot
parent 4e3c3d8c6a
commit 5a6ad9d52a
1 changed files with 1 additions and 3 deletions

View File

@ -108,11 +108,9 @@ CFLAGS += -DHAVE_POWER8
HAVE_POWER8=1
endif
# if we're compiling for release, compile without debug code (-DNDEBUG) and
# don't treat warnings as errors
# if we're compiling for release, compile without debug code (-DNDEBUG)
ifeq ($(DEBUG_LEVEL),0)
OPT += -DNDEBUG
DISABLE_WARNING_AS_ERROR=1
ifneq ($(USE_RTTI), 1)
CXXFLAGS += -fno-rtti