Fix a Clang warning in the new NVPTX backend:
authorChandler Carruth <chandlerc@gmail.com>
Fri, 4 May 2012 21:35:49 +0000 (21:35 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Fri, 4 May 2012 21:35:49 +0000 (21:35 +0000)
In file included from ../lib/Target/NVPTX/VectorElementize.cpp:53:
../lib/Target/NVPTX/NVPTX.h:44:3: warning: default label in switch which covers all enumeration values [-Wcovered-switch-default]
  default: assert(0 && "Unknown condition code");
  ^
1 warning generated.

The prevailing pattern in LLVM is to not use a default label, and instead to
use llvm_unreachable to denote that the switch in fact covers all return paths
from the function.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@156209 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/NVPTX/NVPTX.h

index ebdf423eb23431409fed7d341e58db7a68bbb94f..8ec8eb36b0b2ae6dd86e036715e4700f5e8076ea 100644 (file)
@@ -41,7 +41,6 @@ enum CondCodes {
 
 inline static const char *NVPTXCondCodeToString(NVPTXCC::CondCodes CC) {
   switch (CC) {
-  default: assert(0 && "Unknown condition code");
   case NVPTXCC::NE:  return "ne";
   case NVPTXCC::EQ:   return "eq";
   case NVPTXCC::LT:   return "lt";
@@ -49,6 +48,7 @@ inline static const char *NVPTXCondCodeToString(NVPTXCC::CondCodes CC) {
   case NVPTXCC::GT:  return "gt";
   case NVPTXCC::GE:   return "ge";
   }
+  llvm_unreachable("Unknown condition code");
 }
 
 FunctionPass *createNVPTXISelDag(NVPTXTargetMachine &TM,