[clang] Fix build issue after c474f8f (PR #141954)

Victor Lomuller via cfe-commits cfe-commits at lists.llvm.org
Thu May 29 07:13:50 PDT 2025


https://github.com/Naghasan created https://github.com/llvm/llvm-project/pull/141954

Remove unneeded default label which triggers a warning with -Wcovered-switch-default

>From 5b5bbe5792043881e03db19e9e888bd797052b8e Mon Sep 17 00:00:00 2001
From: Victor Lomuller <victor at codeplay.com>
Date: Thu, 29 May 2025 15:10:55 +0100
Subject: [PATCH] Fix build issue after c474f8f

Remove unneeded default label which triggers a warning with -Wcovered-switch-default
---
 clang/lib/Sema/SemaSPIRV.cpp | 2 --
 1 file changed, 2 deletions(-)

diff --git a/clang/lib/Sema/SemaSPIRV.cpp b/clang/lib/Sema/SemaSPIRV.cpp
index bd72194b905f5..c27d3fed2b990 100644
--- a/clang/lib/Sema/SemaSPIRV.cpp
+++ b/clang/lib/Sema/SemaSPIRV.cpp
@@ -129,8 +129,6 @@ static bool checkGenericCastToPtr(Sema &SemaRef, CallExpr *Call) {
     AddrSpace = SemaRef.LangOpts.isSYCL() ? LangAS::sycl_private
                                           : LangAS::opencl_private;
     break;
-  default:
-    llvm_unreachable("Invalid builtin function");
   }
   Qual.setAddressSpace(AddrSpace);
   Call->setType(SemaRef.getASTContext().getPointerType(



More information about the cfe-commits mailing list