From 3f756fbabe58e2eb03ea0e4f56aac08e4eb59477 Mon Sep 17 00:00:00 2001 From: Reid Kleckner Date: Thu, 1 Nov 2018 19:11:05 +0000 Subject: 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 --- llvm/lib/Transforms/Utils/FunctionComparator.cpp | 2 -- 1 file changed, 2 deletions(-) (limited to 'llvm/lib/Transforms/Utils/FunctionComparator.cpp') 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(TyL)->getBitWidth(), cast(TyR)->getBitWidth()); -- cgit v1.2.3