diff options
| author | Chandler Carruth <chandlerc@gmail.com> | 2012-05-04 21:35:49 +0000 |
|---|---|---|
| committer | Chandler Carruth <chandlerc@gmail.com> | 2012-05-04 21:35:49 +0000 |
| commit | cd3464ee22b48482f80e8d9723348d8fd0a82847 (patch) | |
| tree | 994fd0f2e54f35f98b853859b8ff4e39f11c6be8 /llvm/lib/Target/NVPTX/NVPTX.h | |
| parent | 6781821c010645123a86ee102a9673c8ad460810 (diff) | |
| download | bcm5719-llvm-cd3464ee22b48482f80e8d9723348d8fd0a82847.tar.gz bcm5719-llvm-cd3464ee22b48482f80e8d9723348d8fd0a82847.zip | |
Fix a Clang warning in the new NVPTX backend:
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.
llvm-svn: 156209
Diffstat (limited to 'llvm/lib/Target/NVPTX/NVPTX.h')
| -rw-r--r-- | llvm/lib/Target/NVPTX/NVPTX.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/NVPTX/NVPTX.h b/llvm/lib/Target/NVPTX/NVPTX.h index ebdf423eb23..8ec8eb36b0b 100644 --- a/llvm/lib/Target/NVPTX/NVPTX.h +++ b/llvm/lib/Target/NVPTX/NVPTX.h @@ -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, |
