OSDN Git Service

Fix a -Wcovered-switch-default warning in emitJumpTables.
authorJan Voung <jvoung@chromium.org>
Fri, 31 Jul 2015 04:29:14 +0000 (21:29 -0700)
committerJan Voung <jvoung@chromium.org>
Fri, 31 Jul 2015 04:29:14 +0000 (21:29 -0700)
The Subzero build inside of the LLVM build system turns this on.

BUG=none
R=ascull@google.com

Review URL: https://codereview.chromium.org/1264913005 .

Makefile.standalone
src/IceCfg.cpp

index 233b707..ae9d505 100644 (file)
@@ -159,9 +159,12 @@ CXX := CCACHE_CPP2=yes $(CCACHE) $(CLANG_PATH)/clang++
 SB_CXX := CCACHE_CPP2=yes $(CCACHE) $(PNACL_BIN_PATH)/pnacl-clang++
 SB_TRANSLATE := $(PNACL_BIN_PATH)/pnacl-translate
 
+# Extra warnings that LLVM's build system adds in addition to -Wall.
+LLVM_EXTRA_WARNINGS := -Wcovered-switch-default
+
 BASE_CXXFLAGS := -std=gnu++11 -Wall -Wextra -Werror -fno-rtti \
        -fno-exceptions $(OPTLEVEL) $(ASSERTIONS) -g -pedantic \
-       $(CXX_EXTRA)
+       $(LLVM_EXTRA_WARNINGS) $(CXX_EXTRA)
 
 CXXFLAGS := $(LLVM_CXXFLAGS) $(BASE_CXXFLAGS) $(CXX_DEFINES) $(HOST_FLAGS) \
        $(STDLIB_FLAGS)
index 9e5df99..ef49c15 100644 (file)
@@ -627,9 +627,6 @@ void Cfg::emitJumpTables() {
     for (const InstJumpTable *JumpTable : JumpTables)
       getTarget()->emitJumpTable(this, JumpTable);
   } break;
-  default:
-    llvm::report_fatal_error("Invalid out file type.");
-    break;
   }
 }