diff options
| author | Reid Kleckner <rnk@google.com> | 2018-11-01 19:11:05 +0000 |
|---|---|---|
| committer | Reid Kleckner <rnk@google.com> | 2018-11-01 19:11:05 +0000 |
| commit | 3f756fbabe58e2eb03ea0e4f56aac08e4eb59477 (patch) | |
| tree | a31301c834cccdfe44361c21daf089e0d18d34e7 /llvm/lib/Transforms/Utils/FunctionComparator.cpp | |
| parent | 0a8dc9eb0fe0c1eeae027f6d6d570795b7b6fda6 (diff) | |
| download | bcm5719-llvm-3f756fbabe58e2eb03ea0e4f56aac08e4eb59477.tar.gz bcm5719-llvm-3f756fbabe58e2eb03ea0e4f56aac08e4eb59477.zip | |
Remove unnecessary fallthrough annotation after unreachable
Clang's -Wimplicit-fallthrough implementation warns on this. I built
clang with GCC 7.3 in +asserts and -asserts mode, and GCC doesn't warn
on this in either configuration. I think it is unnecessary. I separated
it from the large mechanical patch (https://reviews.llvm.org/D53950) in
case I am wrong and it has to be reverted.
llvm-svn: 345876
Diffstat (limited to 'llvm/lib/Transforms/Utils/FunctionComparator.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Utils/FunctionComparator.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Utils/FunctionComparator.cpp b/llvm/lib/Transforms/Utils/FunctionComparator.cpp index ef991d715fd..a717d9b7281 100644 --- a/llvm/lib/Transforms/Utils/FunctionComparator.cpp +++ b/llvm/lib/Transforms/Utils/FunctionComparator.cpp @@ -410,8 +410,6 @@ int FunctionComparator::cmpTypes(Type *TyL, Type *TyR) const { switch (TyL->getTypeID()) { default: llvm_unreachable("Unknown type!"); - // Fall through in Release mode. - LLVM_FALLTHROUGH; case Type::IntegerTyID: return cmpNumbers(cast<IntegerType>(TyL)->getBitWidth(), cast<IntegerType>(TyR)->getBitWidth()); |
