[clang] 383057e - [Clang][AArch64] Fix -Wcovered-switch-default warning (SemaARM)

Spencer Abson via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 6 07:23:24 PDT 2024


Author: Spencer Abson
Date: 2024-09-06T14:19:13Z
New Revision: 383057e5a0b53a79926dbdbae4cfcd39bfb44645

URL: https://github.com/llvm/llvm-project/commit/383057e5a0b53a79926dbdbae4cfcd39bfb44645
DIFF: https://github.com/llvm/llvm-project/commit/383057e5a0b53a79926dbdbae4cfcd39bfb44645.diff

LOG: [Clang][AArch64] Fix -Wcovered-switch-default warning (SemaARM)

Added: 
    

Modified: 
    clang/include/clang/Basic/TargetBuiltins.h
    clang/lib/Sema/SemaARM.cpp

Removed: 
    


################################################################################
diff  --git a/clang/include/clang/Basic/TargetBuiltins.h b/clang/include/clang/Basic/TargetBuiltins.h
index 124e353a5c1c47..d0f41b17c154f3 100644
--- a/clang/include/clang/Basic/TargetBuiltins.h
+++ b/clang/include/clang/Basic/TargetBuiltins.h
@@ -236,6 +236,7 @@ namespace clang {
       case Poly128:
         return 128;
       }
+      llvm_unreachable("Invalid NeonTypeFlag!");
     }
   };
 

diff  --git a/clang/lib/Sema/SemaARM.cpp b/clang/lib/Sema/SemaARM.cpp
index 770968cb2d4b46..efde354860de43 100644
--- a/clang/lib/Sema/SemaARM.cpp
+++ b/clang/lib/Sema/SemaARM.cpp
@@ -504,9 +504,6 @@ bool SemaARM::CheckImmediateArg(CallExpr *TheCall, unsigned CheckTy,
         SemaRef.BuiltinConstantArgMultiple(TheCall, ArgIdx, 2))
       return true;
     break;
-  default:
-    llvm_unreachable("Invalid immediate range typeflag!");
-    break;
   }
   return false;
 }


        


More information about the cfe-commits mailing list