commit c2648c2db7f2d0c10ef54393098fe74f65296c45
author Jan Voung <jvoung@chromium.org> Thu Jul 30 21:29:14 2015 -0700
committer Jan Voung <jvoung@chromium.org> Thu Jul 30 21:29:14 2015 -0700
tree ce2a3798afd5beb1c04fc02a6340c25e1158edb4
parent 86df4e9e6d183f07638440afd2c225b485c03917

Fix a -Wcovered-switch-default warning in emitJumpTables.

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

diff --git a/Makefile.standalone b/Makefile.standalone
index 233b707..ae9d505 100644
--- a/Makefile.standalone
+++ b/Makefile.standalone
@@ -159,9 +159,12 @@
 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)

src/IceCfg.cpp

diff --git a/src/IceCfg.cpp b/src/IceCfg.cpp
index 9e5df99..ef49c15 100644
--- a/src/IceCfg.cpp
+++ b/src/IceCfg.cpp
@@ -627,9 +627,6 @@
     for (const InstJumpTable *JumpTable : JumpTables)
       getTarget()->emitJumpTable(this, JumpTable);
   } break;
-  default:
-    llvm::report_fatal_error("Invalid out file type.");
-    break;
   }
 }